-
Notifications
You must be signed in to change notification settings - Fork 100
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
Change /all
to /documents
#306
Conversation
4cd7fa9
to
5f98beb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you think we should add an acceptance test for the redirect? (not sure offhand if this is possible, but if so, it might be able to catch regressions)
Thanks for catching this. I wrote one in my spike branch but forgot to bring it over. ✅ |
Moves the "All Docs" route from
/all
to/documents
to give us a better framework to add new models, e.g., projects, products, teams, that can be browsed. References to the/all
route are now redirected to/documents
.