Skip to content

Refactor Router#132

Merged
rotanid merged 3 commits intomainfrom
router
Aug 5, 2025
Merged

Refactor Router#132
rotanid merged 3 commits intomainfrom
router

Conversation

@MyIgel
Copy link
Member

@MyIgel MyIgel commented Jan 18, 2025

Description

Refactor router to be class based, update navigo based on #36, updated vite to fix annoying warning of empty source maps

Motivation and Context

How Has This Been Tested?

Screenshots/links:

Checklist:

  • My code follows the code style of this project. (CI will test it anyway and also needs approval)
  • My change requires a change to the documentation.
    • I have updated the documentation accordingly.

@MyIgel MyIgel force-pushed the router branch 4 times, most recently from 7ed0899 to 3d1270d Compare January 18, 2025 11:21
@MyIgel
Copy link
Member Author

MyIgel commented Mar 1, 2025

Still a draft as it does not work that way (unstable url changes..)

@maurerle
Copy link
Member

@MyIgel do you know what is still missing here?

maurerle and others added 3 commits August 1, 2025 20:45
this is required for compatibility with client libraries which are imported differently
Bumps [navigo](https://github.com/krasimir/navigo) from 7.1.3 to 8.11.1.
- [Changelog](https://github.com/krasimir/navigo/blob/master/CHANGELOG.md)
- [Commits](https://github.com/krasimir/navigo/commits)

---
updated-dependencies:
- dependency-name: navigo
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@maurerle maurerle marked this pull request as ready for review August 1, 2025 19:17
@maurerle
Copy link
Member

maurerle commented Aug 1, 2025

tested and working - waiting for #202 to be merged

@rotanid rotanid merged commit d01f2bd into main Aug 5, 2025
2 checks passed
@rotanid rotanid deleted the router branch August 5, 2025 01:37
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