Skip to content
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

Initialize routes per Rails::Engine Initializer #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Initialize routes per Rails::Engine Initializer #8

wants to merge 2 commits into from

Conversation

timoschilling
Copy link
Contributor

No description provided.

@jhilden
Copy link
Contributor

jhilden commented Mar 20, 2013

I would like to keep the option for the user to define where in his routes the gem will be mounted. I uess many people would like to have it at /styleguide rather than /smurfville

Could this still be done after your changes?

@timoschilling
Copy link
Contributor Author

not direct, but i can write a configuration for that

@jhilden
Copy link
Contributor

jhilden commented Mar 20, 2013

I think a config option like auto_mounting (default = true) would good. That would leave maximum flexibility for those that want customization.

@timoschilling
Copy link
Contributor Author

No, I think on something like

config.smurfville.mount_at = "/styleguide"

The default is /smurfville, so the User need to do nothing else as gem "smurfville" in the standard case and can optional configure a different endpoint.

Merge this and i build the Configurator today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants