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-2577: Add pdlClient to get name in api for journalforing #8

Merged
merged 1 commit into from
Aug 12, 2024

Conversation

eirikdahlen
Copy link
Collaborator

@eirikdahlen eirikdahlen commented Aug 8, 2024

Har bedt om tilgang til PDL her: https://jira.adeo.no/plugins/servlet/desk/portal/541/FAGSYSTEM-341015
EDIT: Fått tilgang ✅

For å journalføre må vi både legge inn PdlClient, DokarkivClient og CronjobModule. Starter med denne, så kommer det flere PRer etterhvert. For det meste copy+paste fra isarbeidsuforhet.

Ettersom brevmal ikke er klart enda, så tenker jeg å legge cronjobben bak en toggle, og så heller slå den på når vi har gjort klart PDF-generering med riktig mal. I mellomtiden kan vi lage malene i ispdfgen sånn at alt "henger sammen".

Ble litt misvisende branch-navn...

@eirikdahlen eirikdahlen requested a review from a team as a code owner August 8, 2024 13:00
Comment on lines +70 to +73
// Se behandlingskatalog https://behandlingskatalog.intern.nav.no/
// Behandling: Sykefraværsoppfølging: Vurdere behov for oppfølging og rett til sykepenger etter §§ 8-4 og 8-8
private const val BEHANDLINGSNUMMER_HEADER_KEY = "behandlingsnummer"
private const val BEHANDLINGSNUMMER_HEADER_VALUE = "B426"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Har dobbeltsjekket at dette er riktig behandling for denne casen også 👍🏼

Copy link
Contributor

@vetlesolgaard vetlesolgaard left a comment

Choose a reason for hiding this comment

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

Ser bra ut! 🙌

@@ -74,3 +74,7 @@ spec:
value: "dev-gcp.teamsykefravr.istilgangskontroll"
- name: ISTILGANGSKONTROLL_URL
value: "http://istilgangskontroll"
- name: PDL_CLIENT_ID
Copy link
Contributor

Choose a reason for hiding this comment

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

Må legge til en "rule" ovenfor under accessPolicy.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ja, den lå der visst fra en tidligere commit, helt sikkert noe vi kopierte over da vi lagde repoet bare 👍🏼

@eirikdahlen eirikdahlen merged commit 4cbc037 into main Aug 12, 2024
3 checks passed
@eirikdahlen eirikdahlen deleted the IS-2577-journalforing-cronjob branch August 12, 2024 11:52
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