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

IS-2643: Turn off JP-retry in dev #275

Merged
merged 5 commits into from
Dec 17, 2024
Merged

Conversation

geir-waagboe
Copy link
Contributor

Hva har blitt lagt til✨🌈

Skrur av retry av journalføring i dev (i dag står dialogmeldingene og feiler i det uendelige dersom de er sendt inn på personer som mangler aktørid)

@geir-waagboe geir-waagboe requested a review from a team as a code owner December 16, 2024 12:26
}
null
}
val journalpostId = journalpost?.journalpostId ?: "0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Betyr dette at vi lagrer dialogmelding med journalpostId satt til 0 i db i dev? Hadde det vært bedre og gjort dette i de andre appene i dev også (i stedet for å skru av journalføring). Det gjør det kanskje enklere å teste journalføringen i dev 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ja, det er det jeg endte opp med å gjøre her siden journalføringen er et "trinn" på veien mot å ferdigbehandle. Ja, jeg funderte om vi kunne gjort det på denne måten i de andre app'ene også.. Er ikke helt sikker.

Copy link
Contributor

Choose a reason for hiding this comment

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

Da vil vi i hvert fall kunne teste journalføring (og nedstrøms-greier som er avhengig av at journalpostId er satt) uten å måtte enable cronjob og prosessere feilende journalføringer. Kunne i så fall laget en egen feilhåndtering av manglende aktør-id feilen som bare er aktiv i dev og som setter journalpostId til 0.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ja, gjør en tilpasning for dette i de andre PR'ene. Men denne er kanskje ok da?

Copy link
Contributor

@eirikdahlen eirikdahlen left a comment

Choose a reason for hiding this comment

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

Nå blir det vel dev-spesifikk kode ved å gjøre det sånn her, men jeg synes likevel det høres ut som en god ide 👍🏼 Kunne vi dokumentert det noe sted i koden, eller andre steder?

@geir-waagboe
Copy link
Contributor Author

Nå blir det vel dev-spesifikk kode ved å gjøre det sånn her, men jeg synes likevel det høres ut som en god ide 👍🏼 Kunne vi dokumentert det noe sted i koden, eller andre steder?

Ja, legger på en kommentar i hvert fall.

@geir-waagboe geir-waagboe merged commit 570a4b6 into master Dec 17, 2024
4 checks passed
@geir-waagboe geir-waagboe deleted the disable-jp-retry-dev branch December 17, 2024 14:26
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.

3 participants