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(): update url structure to make more sense #1854

Merged
merged 9 commits into from
Apr 8, 2025

Conversation

astoervold
Copy link
Contributor

@astoervold astoervold commented Mar 28, 2025

Oppdatere URL strukturen så den gir mer mening og bruker norsk


Motivasjon

Vi ønsker at slugs skal gi mer mening etter hvordan nettsiden er satt opp med tavler og mapper. Vi ønsker også å matche resten av Entur som bruker norske slugs.

Endringer

  • Lagde ny mappe som heter "oversikt" hvor page.tsx (som lå inni boards) ble flyttet til.
  • Lagde en ny mappe som heter "mapper" hvor [id]-mappen (som lå under boards) ble flyttet til.
  • La til en ny mappe som heter "rediger" under /mapper/[id] og flyttet page.tsx for redigering av en mappe inn dit.
  • Slettet boards-mappen etter å ha flyttet resten av innholdet der det hørte hjemme
  • Endret navnet på /edit til å være /tavler og la inn en ny mappe "rediger" og flyttet page.tsx for redigering av en tavle inn dit.
Strukturendring
image

Sjekkliste for Review

  • At knapper som tidligere ledet til sider fremdeles leder til de riktige sidene

@astoervold astoervold requested a review from emilielr as a code owner March 28, 2025 09:41
@astoervold astoervold mentioned this pull request Mar 31, 2025
1 task
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.

Elsker diagrammet! 🕴

* fix(edit): delete board

* refactor(): rename function for clarity
@astoervold astoervold changed the title feat(url-structure): updated url structure to make more sense feat(): updated url structure to make more sense Apr 1, 2025
@astoervold astoervold changed the title feat(): updated url structure to make more sense feat(): update url structure to make more sense Apr 1, 2025
@astoervold astoervold force-pushed the update-url-structure branch from 225dc57 to 066e9e1 Compare April 3, 2025 06:53
@astoervold astoervold requested a review from emilielr April 3, 2025 07:46
@emilielr
Copy link
Collaborator

emilielr commented Apr 7, 2025

Si fra når du merger denne, da må vi oppdatere URLen i posthog-surveyen vår til å være rediger i stedet for edit

@astoervold astoervold merged commit 4d87f6b into main Apr 8, 2025
3 checks passed
@astoervold astoervold deleted the update-url-structure branch April 8, 2025 06:38
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