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
Redirect deprecated routes #50
Conversation
@@ -0,0 +1,13 @@ | |||
'use strict'; |
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.
what about putting this in the /deprecated folder?
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.
Good idea, I've moved it
|
||
const request = require('supertest'); | ||
|
||
describe('GET /v1/files', function() { |
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.
I'd personally find it easier if this was in the files.js but in a different describe, but I don't have a strong opinion about it
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.
Ah I'm trying to match the filenames in here to the URLs
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.
Yup, and that's fine as well, just wanted to mention my personal preference :P
Looks good! I don't think this should break in any way, files and modules haven't changed |
This redirects file, module, and documentation routes in the same way as bundle routes are redirected.