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

Tidy up admin page routing #204

Open
domdomegg opened this issue Nov 18, 2022 · 0 comments
Open

Tidy up admin page routing #204

domdomegg opened this issue Nov 18, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@domdomegg
Copy link
Member

Currently, I think the routing around the admin pages is quite hacky. It's reach router sitting on top of Gatsby, which doesn't play that nice and adds complexity.

I do think we'd like to keep parameters in the path if possible, so we can share links between people. However, maybe we want to do this a different way - possibly moving this into query parameter routing or similar. Or, figure out how we can do this properly in Gatsby (or Next.js if we want to migrate) in a way that's compatible with our static hosting setup.

Possibly related links:

@domdomegg domdomegg added the enhancement New feature or request label Nov 21, 2022
@domdomegg domdomegg changed the title Fix admin page routing Tidy up admin page routing Nov 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant