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

feat(next): upgrade to next 15.2.3 #1848

Merged
merged 1 commit into from
Apr 11, 2025
Merged

feat(next): upgrade to next 15.2.3 #1848

merged 1 commit into from
Apr 11, 2025

Conversation

emilielr
Copy link
Collaborator

@emilielr emilielr commented Mar 24, 2025

Oppdatere til Next 15.2.3


Motivasjon

Det er en sårbarhet som er funnet i versjonen vi har som handler om at man kan bypasse autorisasjon i middleware: se her. Vi har ikke middleware, men det kan være fint å oppgradere likevel slik at vi patcher sårbarheten og slik at vi ikke "glemmer" om vi plutselig skal ha autorisering i middleware i fremtiden.

Release notes til Next 15.2: https://nextjs.org/blog/next-15-2
Ingenting her som skal tilsi en forandring for vår del.

Endringer

  • Oppgradert til Next 15.2.3
  • Linting av package.json filen

Sjekkliste for Review

  • Sjekk at tavlevisningen ikke er påvirket i Browserstack (gjerne test litt forskjellige versjoner + nettlesere)
  • Klikk deg litt rundt i adminløsningen og se at ting ikke er rart eller forandret på

@emilielr emilielr changed the title chore(): upgrade to next 15.2.3 feat(next): upgrade to next 15.2.3 Apr 1, 2025
Copy link
Contributor

@siljemoksnes siljemoksnes left a comment

Choose a reason for hiding this comment

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

Alt funker som forventet! Altså, fungerer med yarn dev for tavler i Chrome-versjon fra 64 og høyere, og yarn build-start for versjoner ned til 49. Fungerer også i diverse andre browsere.

Får opp denne feilmeldingen når jeg trykker "publiser tavle" etter å ha lagt til nye verdier. Kan godt være at det er forventet i dev/localhost, eller at det er jeg som er ute i litt funky business 🕺
image

@emilielr
Copy link
Collaborator Author

emilielr commented Apr 11, 2025

Alt funker som forventet! Altså, fungerer med yarn dev for tavler i Chrome-versjon fra 64 og høyere, og yarn build-start for versjoner ned til 59. Fungerer også i diverse andre browsere.

Får opp denne feilmeldingen når jeg trykker "publiser tavle" etter å ha lagt til nye verdier. Kan godt være at det er forventet i dev/localhost, eller at det er jeg som er ute i litt funky business 🕺 image

Publiser tavle funker ikke med mindre du også spinner opp backenden vår, vet ikke om du gjorde det? Og med yarn build-start, så funker den vel ned til 49? :thinkies:

Chrome 49 hos meg:
image

@siljemoksnes
Copy link
Contributor

Alt funker som forventet! Altså, fungerer med yarn dev for tavler i Chrome-versjon fra 64 og høyere, og yarn build-start for versjoner ned til 59. Fungerer også i diverse andre browsere.
Får opp denne feilmeldingen når jeg trykker "publiser tavle" etter å ha lagt til nye verdier. Kan godt være at det er forventet i dev/localhost, eller at det er jeg som er ute i litt funky business 🕺 image

Publiser tavle funker ikke med mindre du også spinner opp backenden vår, vet ikke om du gjorde det? Og med yarn build-start, så funker den vel ned til 49? :thinkies:

Chrome 49 hos meg: image

Aah ja det må være derfor! Ja, en liten skrivefeil der, ned til 49 🤠

@emilielr emilielr merged commit d68b596 into main Apr 11, 2025
3 checks passed
@emilielr emilielr deleted the upgrade-next branch April 11, 2025 09:15
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