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

Dev next1 from master merge #6422

Merged
merged 28 commits into from
Jul 24, 2024
Merged

Dev next1 from master merge #6422

merged 28 commits into from
Jul 24, 2024

Conversation

josstn
Copy link
Collaborator

@josstn josstn commented Jul 24, 2024

Oppdateringer frå master til dev-next1 branch.

josstn and others added 28 commits July 3, 2024 14:43
* nais deploy konfigurasjon for dev-next1 oppretta.

name, ingress og replicas konfigurasjon for nais dev deploy trekt ut i separate filer.

dev.yaml inneheld samme konfigurasjonsverdier som før var hardkoda i dev-fss-k9saksbehandling.yml, for normal utrulling til Q

dev-next1.yaml inneheld konfigurasjonsverdier for utrulling til ny dev-next1 instans.

* Build and deploy dev-next1 workflow oppretta.

Denne vil ved push til dev-next1 branch bygge og rulle ut instans av k9-sak-web til k9.dev-next1.intern.navn.no, slik at meir eksperimentelle ting kan testast der før det går til master.

* Fiks ingress url til gyldig verdi.
Hadde brukt makslengde verdi frå FritekstbrevinnholdDto server har, men maks lengde er faktisk ulik alt etter om malen som er valgt er av type som større fritekst og tittel, eller berre fritekst.

Endre til å bruke BestillBrevDto makslengde når mal ikkje støtter tittel,og FritekstbrevinnholdDto sin makslengde når mal støtter tittel.

Endre også slik at fritekst prop ikkje blir satt (sendt til server) når det er brev med tittel og tekst, sidan den då vil kunne bli for lang.
Bumps [@navikt/k9-sak-typescript-client](https://github.com/navikt/k9-sak) from 1.0.20240620093339 to 1.0.20240708134619.
- [Commits](https://github.com/navikt/k9-sak/commits)

---
updated-dependencies:
- dependency-name: "@navikt/k9-sak-typescript-client"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps the patch group with 6 updates in the / directory:

| Package | From | To |
| --- | --- | --- |
| [@playwright/test](https://github.com/microsoft/playwright) | `1.45.0` | `1.45.1` |
| [postcss](https://github.com/postcss/postcss) | `8.4.38` | `8.4.39` |
| [typescript](https://github.com/Microsoft/TypeScript) | `5.5.2` | `5.5.3` |
| [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) | `5.3.2` | `5.3.3` |
| [@editorjs/header](https://github.com/editor-js/header) | `2.8.1` | `2.8.5` |
| [@editorjs/paragraph](https://github.com/editor-js/paragraph) | `2.11.5` | `2.11.6` |



Updates `@playwright/test` from 1.45.0 to 1.45.1
- [Release notes](https://github.com/microsoft/playwright/releases)
- [Commits](microsoft/playwright@v1.45.0...v1.45.1)

Updates `postcss` from 8.4.38 to 8.4.39
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss@8.4.38...8.4.39)

Updates `typescript` from 5.5.2 to 5.5.3
- [Release notes](https://github.com/Microsoft/TypeScript/releases)
- [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml)
- [Commits](microsoft/TypeScript@v5.5.2...v5.5.3)

Updates `vite` from 5.3.2 to 5.3.3
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v5.3.3/packages/vite)

Updates `@editorjs/header` from 2.8.1 to 2.8.5
- [Commits](https://github.com/editor-js/header/commits)

Updates `@editorjs/paragraph` from 2.11.5 to 2.11.6
- [Commits](https://github.com/editor-js/paragraph/commits)

---
updated-dependencies:
- dependency-name: "@playwright/test"
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: patch
- dependency-name: postcss
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: patch
- dependency-name: typescript
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: patch
- dependency-name: vite
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: patch
- dependency-name: "@editorjs/header"
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: patch
- dependency-name: "@editorjs/paragraph"
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jostein Stuhaug <135307852+josstn@users.noreply.github.com>
* Oppdatert README.md

Fjerna utdaterte ting, la inn litt ny nyttig info.

* Oppdatert README.md

Litt forbedring av teksten.

* Oppdatert README.md

Litt info om bygg og deploy av prosjektet.
* Rydding i packages/form. Ingen funksjonell endring.

- Fjern unødvendig prop-types dependency
- Endre gjenverande jsx til tsx

* Rydding i packages/prop-types. Ingen funksjonell endring.

Fjern ubruk behandlingIListePropType.jsx

* Rydding i packages/prop-types. Ingen funksjonell endring.

Fjern ubrukt arbeidsforholdPropType.jsx og uttaksresultatPropType.jsx

* Rydding i packages/prop-types. Ingen funksjonell endring.

Fjern ubrukt beregningresultatMedUttaksplanPropType.jsx og fagsakPropType.jsx

* Rydding i packages/prop-types. Ingen funksjonell endring.

Fjern ubrukt navAnsattPropType.jsx, personopplysningPropType.jsx og rettighetPropType.jsx

* Rydding i packages/prop-types. Ingen funksjonell endring.

Fjern ubrukt stonadskontoPropType.jsx.
* 10 dager ttl på dev-next1 branch deployment.

Sidan dev-next1 branch deployment kanskje ikkje blir brukt heile tida legger eg til "time to live" på ti dager på app spec. Det vil seie at app blir sletta etter 10 dager viss det ikkje har skjedd nokon ny bygg og deploy av dev-next1 branch på den tida.

Så slepper vi å bruke ressurser på å ha den køyrande ubrukt.

* Info om ttl på dev-next1 lagt til i README.md
* Storbritannia er ikke et EØS-land. Fikser visning av årsak for Sveits

* Storbritannia er ikke et EØS-land. Fikser visning av årsak for Sveits

* Lint fiks.

* tester for utenlandsopphold-visning

* kosovo fix

---------

Co-authored-by: Jostein Stuhaug <jostein.stuhaug@nav.no>
Slik at den stemmer betre.
* Github pipeline workflow uten issue for deployment.

Erstatter bruken av issue med kommentering for å styre utrulling av endringer med å køyre utrulling til Q direkte i workflow, og bruke environment klausul med godkjenning ved utrulling til prod.

Legger og inn tagging av prodsatte commits, samme tag som docker image som blir rulla ut har. Legger og inn ein latest_prod tag som alltid peikar på den commit som sist vart rulla ut til prod.

Med dette får vi og printa ut url for å enkelt sjå kva endringer som vil bli rulla ut, eller har blitt rulla ut på ei gitt workflow køyring.

* Marker utrulling til dev med environment og.

Slik at det kjem fram i deployments oversikt på repoet.
* Div test/build dependencies oppdatert.

* i18n-iso-countries oppdatert.

* Fjerner webpack-merge avhengighet.

Kan bruke standard object merge istadenfor.

* Oppdatert uuid til versjon 10.

* Oppdatert vitest til versjon 2.

Einaste breaking change som vart vurdert til å kunne ha konsekvens for oss er vitest-dev/vitest#5749, men fant ingen tilfeller i koden der mock results er sjekka.
Lar patch og minor oppdateringer gå i samme gruppe.

Legger og opp til ei gruppering for storybook major oppdateringer, men at vanlege går i hovedgruppa.
# Conflicts:
#	.github/workflows/promote-command.yml
#	deploy/dev-next1.yaml
#	package.json
#	packages/fakta-etablert-tilsyn/package.json
#	packages/fakta-inntektsmelding/package.json
#	packages/fakta-medisinsk-vilkår/package.json
#	packages/fakta-medisinsk-vilkår/src/ui/components/medisinsk-vilkår/MedisinskVilkår.tsx
#	packages/fakta-om-barnet/package.json
#	packages/fakta-omsorgen-for/package.json
#	packages/fakta-utenlandsopphold/src/Utenlandsopphold.tsx
#	packages/prop-types/src/arbeidsforholdPropType.jsx
#	packages/prop-types/src/behandlingIListePropType.jsx
#	packages/prop-types/src/beregningresultatMedUttaksplanPropType.jsx
#	packages/prop-types/src/fagsakPropType.jsx
#	packages/prop-types/src/personopplysningPropType.jsx
#	packages/prosess-omsorgsdager/package.json
#	packages/prosess-uttak/package.json
#	packages/prosess-vedtak/package.json
#	packages/sak-aktor/src/AktorSakIndex.stories.tsx
#	packages/sak-aktor/src/AktorSakIndex.tsx
#	packages/sak-aktor/src/components/AktoerGrid.tsx
#	packages/sak-app/src/aktoer/AktoerIndex.tsx
#	packages/sak-app/src/fagsak/FagsakIndex.spec.tsx
#	packages/v2/gui/src/sak/meldinger/FritekstInput.tsx
#	packages/v2/gui/src/sak/meldinger/Messages.tsx
#	yarn.lock
Denne bugfiks frå master forsvant i merge resolve sidan eg var usikker på kva som var rett då.
I Utenlandsopphold.tsx er prop erstatta med bruk av context.

Prop vart brukt til å sende inn mock data, så må finne ein anna måte å gjere dette på, eller endre tilbake til å bruke prop for utenlandsopphold årsak info brukt der.
I Utenlandsopphold.tsx er prop erstatta med bruk av context.

Legger inn render wrapperfunksjon som legger til nødvendige mockdata i context rundt komponent som testast.
@josstn josstn changed the base branch from master to dev-next1 July 24, 2024 12:34
@josstn josstn marked this pull request as ready for review July 24, 2024 12:40
@josstn josstn requested a review from a team as a code owner July 24, 2024 12:40
@josstn josstn merged commit 14795bc into dev-next1 Jul 24, 2024
5 checks passed
@josstn josstn deleted the dev-next1-from-master-merge branch July 24, 2024 12:41
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.

4 participants