Skip to content

Credential presentation notice#87

Merged
kskaret merged 1 commit intomainfrom
credential-presentation-notice
Feb 2, 2026
Merged

Credential presentation notice#87
kskaret merged 1 commit intomainfrom
credential-presentation-notice

Conversation

@kskaret
Copy link
Contributor

@kskaret kskaret commented Oct 15, 2025

No description provided.

@kskaret kskaret force-pushed the credential-presentation-notice branch from 1732686 to 606b32e Compare November 11, 2025 14:27
@kskaret kskaret requested a review from eirikolberg November 11, 2025 14:27
@kskaret kskaret force-pushed the credential-presentation-notice branch from 85f58be to 461c97d Compare November 12, 2025 09:46
@kskaret
Copy link
Contributor Author

kskaret commented Nov 12, 2025

Jeg har gjort en del endringer her nå.
Introdusert støtte for både DCQL og "simple" query, valgfritt.

Kan du ta en titt @eirikolberg ?

@kskaret kskaret force-pushed the credential-presentation-notice branch from da6b8b6 to 3e41b21 Compare November 27, 2025 15:00
@kskaret
Copy link
Contributor Author

kskaret commented Nov 27, 2025

@eirikolberg Nå tenker jeg denne er klar for code review igjen.
Det hadde vært fint å kunne tatt en merge og release av denne så det blir enkelt å ta den i bruk i de andre modulene.
Status:

  • Jeg har fortsatt beholdt "SimpleQuery" varianten. Grunnen til det er at det er bare den som virker foreløpig i Bevisinnhenter (det mappes til PresentationExchange). Men så fort vi får DCQL til å virke med feks. iGrant wallet tror jeg det er fornuftig å kun støtte det.
  • Jeg har prøvd å gjøre det så enkelt å som mulig å instansiere en DCQL-request vha tre dedikerte factory-metoder. En for hver av jwt_vc, sd_jwt og mdoc. Som sikrer at de får Meta-objektet populert på riktig måte, og hvor man ikke trenger å forholde seg til verdier som ofte uansett er null, som credential_sets.

@kskaret kskaret marked this pull request as ready for review November 28, 2025 08:55
@kskaret kskaret force-pushed the credential-presentation-notice branch from 3e41b21 to 2e632a5 Compare January 20, 2026 07:34
Comment on lines +153 to +158
<opening-receipt xmlns="http://api.digipost.no/schema/datatypes">
<group>aarsoppgave-bedriftAS-2025</group>
</opening-receipt>

<opening-receipt-accepted xmlns="http://api.digipost.no/schema/datatypes"/>

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Her har det skjedd en automatisk omrokkering. Jeg har skjønt på Eivind at det er best om vi flytter den tilbake der den var, slik at det ikke virker som vi har endret noe i denne branchen.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Disse havner i alfabetisk rekkefølge, fordi de sorteres under generering. Se DocumentationStructureBuilder:

  public static <T> Stream<ComplexType> buildTypeStructure(Set<Class<? extends T>> classes, Function<Class<? extends T>, T> getExample) {
        return classes.stream().map(getTypeInfoWithExample(getExample)).sorted();
    }

Så dette ser riktig ut slik det er generert her. Jeg vet ikke hvorfor det har blitt sjekket inn i en annen rekkefølge men jeg foreslår at vi endrer det nå.
@eivinhb

Comment on lines +459 to +492
## OpeningReceipt

To open the document the user must accept to send an opening receipt

### Fields

|Name|Type|Required|Description|
|----|----|--------|-----------|
|group|String|yes|This is the group identifier for the opening receipt|

### XML

```xml
<opening-receipt xmlns="http://api.digipost.no/schema/datatypes">
<group>aarsoppgave-bedriftAS-2025</group>
</opening-receipt>
```

## OpeningReceiptAccepted

The document has been opened, and the opening receipt has been accepted and sent.

### Fields

|Name|Type|Required|Description|
|----|----|--------|-----------|


### XML

```xml
<opening-receipt-accepted xmlns="http://api.digipost.no/schema/datatypes"/>
```

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Samme innspill her om autmoatisk omrokkering. Flytt tilbake.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Samme kommentar som over. Generatoren legger de automatisk i alfabetisk rekkefølge, så denne rekkefølgen er riktig

@kskaret kskaret force-pushed the credential-presentation-notice branch from 40f0576 to 2e632a5 Compare January 22, 2026 15:16
Denne finnes i to varianter, enten SimpleQuery eller DcqlQuery.
@kskaret kskaret force-pushed the credential-presentation-notice branch from a9a23e3 to ca19af7 Compare January 26, 2026 09:56
@kskaret kskaret merged commit 5d76146 into main Feb 2, 2026
4 checks passed
@kskaret kskaret deleted the credential-presentation-notice branch February 2, 2026 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants