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

Fjerner at eksternReferanseId blir satt når vi oppretter journalposte… #1245

Merged
merged 1 commit into from
Mar 16, 2021

Conversation

AnjaAalerud
Copy link
Contributor

…r for utgående brev. Den ble satt til behandlinguuid. Feltet krever en unik id for forsendelsen. Den settes for å kunne brukes til sporing gjennom verdikjeden. Det vil forekomme flere forsendelser for samme behandling. Vi sporer dette tilbake til verdikjeden på andre måter, feks historikk-innslag.

…r for utgående brev. Den ble satt til behandlinguuid. Feltet krever en unik id for forsendelsen. Den settes for å kunne brukes til sporing gjennom verdikjeden. Det vil forekomme flere forsendelser for samme behandling. Vi sporer dette tilbake til verdikjeden på andre måter, feks historikk-innslag.
@AnjaAalerud AnjaAalerud requested a review from a team as a code owner March 16, 2021 13:19
@AnjaAalerud AnjaAalerud requested review from a team and haldersaikat March 16, 2021 13:19
@@ -26,7 +26,7 @@
private BehandlingTema behandlingstema;
private Bruker bruker;
private List<Dokument> dokumenter;
private String eksternReferanseId;
private String eksternReferanseId = null;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Noe usikker på denne her. Kanskje er det bedre å generere en random UUID istf null? @jolarsen hva mener du?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Formidling kjører uten eksternReferanseId og det er OK der. Det er ikke påkrevd,
Har mest verdi for innkommende dokument (Inntektsmelding).
For utgående brev er det mest verdifullt at det er knyttet til sak gjennom journalføring

Men begge steder kan man kutte ut dette. Arkiv bør deserialisere til null
@JsonProperty("eksternReferanseId")
private String eksternReferanseId = null;

@@ -26,7 +26,7 @@
private BehandlingTema behandlingstema;
private Bruker bruker;
private List<Dokument> dokumenter;
private String eksternReferanseId;
private String eksternReferanseId = null;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Formidling kjører uten eksternReferanseId og det er OK der. Det er ikke påkrevd,
Har mest verdi for innkommende dokument (Inntektsmelding).
For utgående brev er det mest verdifullt at det er knyttet til sak gjennom journalføring

Men begge steder kan man kutte ut dette. Arkiv bør deserialisere til null
@JsonProperty("eksternReferanseId")
private String eksternReferanseId = null;

@mrsladek mrsladek merged commit d1043e8 into master Mar 16, 2021
@mrsladek mrsladek deleted the Fjerne-eksternRefId-OpprettJournalpost branch March 16, 2021 14:56
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.

None yet

3 participants