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
Organizes Controllers Helper Methods #357
Conversation
- Moves redirect into the Helpers subdirectory in controller - Updates require path - Renames Helper::Tags to Helpers::CSRF
@@ -1,5 +1,5 @@ | |||
module Amber::Controller::Helpers | |||
module Tag | |||
module CSRF |
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.
Probably name the file csrf.cr as well if you're changing the name of the module.
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.
See my comment but looks good.
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.
Agree with @elorest on renaming the file to match the module. LGTM.
* Organizes Controllers Helper Methods - Moves redirect into the Helpers subdirectory in controller - Updates require path - Renames Helper::Tags to Helpers::CSRF * Rename tags.cr to csrf.cr * Remove include RedirectMethods
* Organizes Controllers Helper Methods - Moves redirect into the Helpers subdirectory in controller - Updates require path - Renames Helper::Tags to Helpers::CSRF * Rename tags.cr to csrf.cr * Remove include RedirectMethods Former-commit-id: d5f143d
Description of the Change
Alternate Designs
Benefits
Possible Drawbacks