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-2300: internarbeidsflate-decorator-v3 #211

Merged
merged 3 commits into from
Sep 16, 2024

Conversation

geir-waagboe
Copy link
Contributor

Har ikke funnet ut av problemet med to to dialogene som dukker opp, men fungerer bra ellers.

@geir-waagboe
Copy link
Contributor Author

Kan det være en eller annen avhengighet som har blitt for gammel (dvs som forårsaker at det dukker advarsler fra dekoratoren)?

@andersrognstad
Copy link
Contributor

Kan det være en eller annen avhengighet som har blitt for gammel (dvs som forårsaker at det dukker advarsler fra dekoratoren)?

Hm, kan undersøke litt 🕵️‍♂️

const host = "syfomodiaperson";
const path = `/sykefravaer/${nyttFnr}`;
window.location.href = fullNaisUrlDefault(host, path);
aktivBruker.mutate(nyttFnr, {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tror dette har vært brukket en stund (men kanskje ikke så mange som hopper til syfomodiaperson fra finnfastlege).

@andersrognstad
Copy link
Contributor

andersrognstad commented Sep 13, 2024

Kan det være en eller annen avhengighet som har blitt for gammel (dvs som forårsaker at det dukker advarsler fra dekoratoren)?

Virker som det er noe med koblingen til modiacontextholder her, siden man også får feilmelding i toppen (og 400-feil i network-fanen) om man trykker på Behold/Endre i modalene. Men klarer ikke se noe forskjell i configen der sammenlignet med navikt/syfomoteoversikt#213 🤔
Eller... 400-feilen kommer fordi man sender inn undefinet/ukjent fnr/enhet til modiacontextholder-apiet når man trykker i modalene, så feilen er at modalene vises :/

@andersrognstad
Copy link
Contributor

Kan det være en eller annen avhengighet som har blitt for gammel (dvs som forårsaker at det dukker advarsler fra dekoratoren)?

Virker som det er noe med koblingen til modiacontextholder her, siden man også får feilmelding i toppen (og 400-feil i network-fanen) om man trykker på Behold/Endre i modalene. Men klarer ikke se noe forskjell i configen der sammenlignet med navikt/syfomoteoversikt#213 🤔 Eller... 400-feilen kommer fordi man sender inn undefinet/ukjent fnr/enhet til modiacontextholder-apiet når man trykker i modalene, så feilen er at modalene vises :/

Virker som det løste seg med Aksel-oppgradering 👀 Lager en PR på det!

@geir-waagboe geir-waagboe force-pushed the internflateecorator-upgrade branch from 847ee56 to d826743 Compare September 16, 2024 08:20
@geir-waagboe
Copy link
Contributor Author

Kan det være en eller annen avhengighet som har blitt for gammel (dvs som forårsaker at det dukker advarsler fra dekoratoren)?

Virker som det er noe med koblingen til modiacontextholder her, siden man også får feilmelding i toppen (og 400-feil i network-fanen) om man trykker på Behold/Endre i modalene. Men klarer ikke se noe forskjell i configen der sammenlignet med navikt/syfomoteoversikt#213 🤔 Eller... 400-feilen kommer fordi man sender inn undefinet/ukjent fnr/enhet til modiacontextholder-apiet når man trykker i modalene, så feilen er at modalene vises :/

Virker som det løste seg med Aksel-oppgradering 👀 Lager en PR på det!

Har oppdatert PR'en nå.

skipModal: true,
appName: "Sykefraværsoppfølging",
fetchActiveEnhetOnMount: false,
onEnhetChanged: (enhetId?: string | null, enhet?: Enhet) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Tror det skal gå an å fjerne parametrene herfra slik at vi slipper warningen under

@geir-waagboe geir-waagboe merged commit 0e3e48b into master Sep 16, 2024
4 checks passed
@geir-waagboe geir-waagboe deleted the internflateecorator-upgrade branch September 16, 2024 12: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.

2 participants