-
Notifications
You must be signed in to change notification settings - Fork 72
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
Correct requests to api.perma.cc that don't specify a route #3174
Correct requests to api.perma.cc that don't specify a route #3174
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #3174 +/- ##
========================================
Coverage 84.22% 84.22%
========================================
Files 52 52
Lines 5591 5592 +1
========================================
+ Hits 4709 4710 +1
Misses 882 882
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
This ticket is ready to review :) |
Are there unit tests to write for this feature? |
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.
These redirects are already working correctly, I think, or at least they do for me. What symptom or issue are you seeing that this PR addresses?
This PR tries to address the scenario where If a developer accidentally |
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.
LGTM. Thanks!
No description provided.