-
Notifications
You must be signed in to change notification settings - Fork 57
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
Added the optional locale scope to the config/routes.rb file. Added a… #58
Conversation
… configuration variable called localize_routes that users can enable (disabled by default). This should prevent conflicts with any installations using a parent path that is not a locale (e.g. /blog/contact-us). Also updated the README to inform the users about the new option and incremented the version number.
@@ -1,3 +1,3 @@ | |||
module ContactUs | |||
VERSION = "1.1.0" | |||
VERSION = "1.1.1" |
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.
Freeze mutable objects assigned to constants.
I don't think the Travis failures are due to these changes. |
…ed strings and <1.9 style hash syntax
@@ -23,6 +23,10 @@ module ContactUs | |||
mattr_accessor :parent_mailer | |||
@@parent_mailer = "ActionMailer::Base" | |||
|
|||
# allows for a locale to appear in the path |
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.
Trailing whitespace detected.
…ed strings and <1.9 style hash syntax
…ed strings and <1.9 style hash syntax
@briri awesome I've made some updates to fix the broken build. Could you please update to the latest master to get the fixes, you can remove the version bump that I'll handle, and cleanup the extra/trailing whitespace warnings. Thank you. |
All set JDutil. I rebased and cleaned up the remaining houndci warnings. I also removed the version number change I had made. Thanks! |
Thanks @briri I've release version 1.2.0 with this included. |
… configuration variable called localize_routes that users can enable (disabled by default). This should prevent conflicts with any installations using a parent path that is not a locale (e.g. /blog/contact-us). Also updated the README to inform the users about the new option and incremented the version number.