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

EY-4963 Sperre endringer på aktivitetspliktsvurderingsbrev via brev r… #7045

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sebassonav
Copy link
Contributor

@sebassonav sebassonav commented Feb 16, 2025

…outes

  • og i frontend fjerne knapper for den

TODO: obs må sjekke db om brev uten kobling til oppgave er opprettet
kan ikke merges før de manuelt opprettede brevene er ferdigbehandliet

sjekk disse:

select b.id from brev b
INNER JOIN hendelse h ON b.id = h.brev_id
where brevkoder IN ('OMSTILLINGSSTOENAD_AKTIVITETSPLIKT_INFORMASJON_10MND_INNHOLD', 'OMSTILLINGSSTOENAD_AKTIVITETSPLIKT_INFORMASJON_4MND_INNHOLD')
AND h.id IN (
SELECT DISTINCT ON (h2.brev_id) h2.id
FROM hendelse h2
WHERE h2.brev_id = b.id
ORDER BY h2.brev_id, h2.opprettet DESC)
AND h.status_id IN ('OPPDATERT', 'OPPRETTET');

i select * from aktivitetsplikt_brevdata where brev_id IN ...

…outes

* og i frontend fjerne knapper for den
Copy link
Contributor

@oyvindsh oyvindsh left a comment

Choose a reason for hiding this comment

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

Opprettelsen av det manuelle brevet burde vel også fjernes fra frontend -- og så lurer jeg på om vi i praksis bruker denne routen for de brevene via behandling, og dermed blokkerer alle endringer også de som er en del av flyten?

@sebassonav sebassonav requested a review from oyvindsh February 17, 2025 10:37
@sebassonav
Copy link
Contributor Author

Er noen brev under behandling så må starte med denne #7058

@sebassonav
Copy link
Contributor Author

update: må fortsatt vente litt. Masse brev igjen

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