Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Etablere "correspondence" som en type dialog #233

Closed
6 tasks done
Tracked by #33 ...
elsand opened this issue Nov 9, 2023 · 4 comments
Closed
6 tasks done
Tracked by #33 ...

Etablere "correspondence" som en type dialog #233

elsand opened this issue Nov 9, 2023 · 4 comments
Assignees

Comments

@elsand
Copy link
Member

elsand commented Nov 9, 2023

Introduksjon

For å kunne skille på "post", altså frittstående meldinger eller korrespondanse, innføres en nytt felt som indikerer dette.

Beskrivelse

Noen meldinger sendes i starten av, underveis eller i slutten av en dialog som inneholder konkrete oppgaver til mottakeren. Disse anses som bare en andel av en "dialog", og det er behov for å kunne skille på disse fra frittstående meldinger som ikke har noen konkrete oppgaver knyttet til seg, altså ikke er en del av en dialog. Denne issuen omfatter å bygge støtte for sistnevnte.

Ved å introdusere en mekanisme for indikere "correspondence", kan vi ha annen håndtering av disse i både frontend og backend. Sentrale egenskaper for en correspondence.

  • Correspondence er "fire-and-forget"; de kan typisk ikke endres etter opprettelse.
  • Correspondence-tjenesten vil ha behov for å kunne legge inn innslag i aktivitetshistorikken ifm lesebekreftelse, og vil kunne trenge å slette den. På sikt vil det kunne være aktuelt med andre endringer av dialogen.
  • Tjenesteeieren skal ikke kunne "gå bakveien" gjennom Dialogporten-API-ene for å opprette eller endre på correspondence-type dialoger - her skal correspondence-tjenesten ha eksklusiv tilgang.

Implementasjon

  • Utvide ResourceType i ressursregister med type Correspondence - Skilt ut i Utvider ResourceType i RR med type for "Correspondence" (el.l.) #723

  • Hvis det blir forsøkt opprettet en dialog med ref til en ressurs som er Correspondence i RR, kreves et spesielt scope (digdir:dialogporten.correspondence). Correspondence-tjenesten vil også ha scopet som definert i Legge inn admin-scope for opprettelse av dialoger uavhengig av ressurs-eier #272, som sjekkes uavhengig.

  • DialogEntity utvides med et nullable felt ServiceResourceType, som populeres ved opprettelse utfra verdi tjenesteressursen har i RR på opprettelsestidspunktet. Skal i første omgang kun populeres hvis ResourceType er Correspondence (kan ikke endres)

  • POST fra tjenesteeier-API skal gi 403 Forbidden hvis oppgitt tjenesteressurs har ServiceResourceType lik Correspondence uten at digdir:dialogporten.correspondence er oppgitt som scope-claim

  • PUT/PATCH/DELETE på tjenesteeier-API skal gi 403 Forbidden hvis den aktuelle dialogen har ServiceResourceType lik Correspondence uten at digdir:dialogporten.correspondence er oppgitt som scope-claim

  • ServiceResourceType skal kun eksponeres i GetDialogDto og SearchDialogDto i både EU og SO

Oppgaver

Preview Give feedback
@elsand elsand changed the title Innføre "correspondence"-verdi på status-enum Etablere "correspondence" som en type dialog Nov 9, 2023
@elsand elsand added the good first issue Good for newcomers label Nov 9, 2023
@elsand elsand removed the good first issue Good for newcomers label Jan 3, 2024
@elsand elsand closed this as completed Jan 11, 2024
@elsand elsand reopened this Apr 15, 2024
@elsand elsand added this to the Pilotproduksjon milestone Apr 18, 2024
@knuhau knuhau self-assigned this Apr 30, 2024
@oskogstad oskogstad self-assigned this May 6, 2024
oskogstad added a commit that referenced this issue May 10, 2024
## Description
Adds a new field for resource type (maps from Altinn Resource Registry)
Now supporting type Correspondence
Correspondence dialogs cannot be changed or deleted without the scope `digdir:dialogporten.correspondence`, and cannot have the field `Progress` set.
DTOs have been updated with  the field ServiceResourceType

## Related Issue(s)

- #233 

## Verification

- [x] **Your** code builds clean without any errors or warnings
- [x] Manual testing done (required)
- [ ] Relevant automated test added (if you find this hard, leave it and
we'll help out)

## Documentation

- [ ] Documentation is updated (either in `docs`-directory, Altinnpedia
or a separate linked PR in
[altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if
applicable)
@oskogstad
Copy link
Collaborator

Moving e2e tests to separate issue, can't complete yet since RR does not have resource type "Correspondence" yet.
(And the correspondence service needs to be up as well?)

Making a few unit tests for now.

@elsand
Copy link
Member Author

elsand commented May 13, 2024

@oskogstad @knuhau Has there been any dialog with melding/formidling about this?

@oskogstad
Copy link
Collaborator

@elsand Not that I'm aware of, regarding anything specific?

@knuhau
Copy link
Collaborator

knuhau commented May 13, 2024

@oskogstad @knuhau Has there been any dialog with melding/formidling about this?

Still no response. The broker team currently has a PR in review that adds another new type (BrokerService) and restructures how the enum values are set. We are creating another PR now that builds on this and adds CorrespondenceService as well. Will ask for reviews from melding/formidling to make sure we are on the same page.

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

No branches or pull requests

3 participants