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

DIGG-512: Initital update to update paths to readable url #550

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

MikaMunterud
Copy link
Contributor

@MikaMunterud MikaMunterud commented Jan 3, 2025

Pull Request Description

Changing urls to be more "readable" if a readable url exists for specifications, concepts and terminology.

Also fixing url paths for sandbox environment.

Adding a redirect logic for old urls in case they have been saved/ bookmarked.

Have to use useEffect because, static props, server props or requests to an internal api route doesn't work when the code is behind a firewall.

Checklist

  • Changes are limited to a single goal (avoid scope creep)
  • I confirm that I have read any Contribution and Development guidelines (CONTRIBUTING and DEVELOPMENT) and are following their suggestions.
  • I confirm that I wrote and/or have the right to submit the contents of my Pull Request, by agreeing to the Developer Certificate of Origin, (adding a 'sign-off' to my commits).

@MikaMunterud MikaMunterud merged commit 224146b into dev Jan 7, 2025
1 check passed
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