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

exploration of issue #5 #7

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

exploration of issue #5 #7

wants to merge 10 commits into from

Conversation

coldnebo
Copy link

@coldnebo coldnebo commented Sep 5, 2014

searching for the root cause to #5 led me to a variety of things as the commit log shows.

you may want to cherrypick or advise on parts that make sense (like relative_url_root support) and others that don't.

the time this Engine runs, so I moved it into an :after block.

This allows webrick to be detected.

Also, the base_urls are being cached on SwaggerYard init, so they were ignoring
SwaggerYard.config block in the Rails app initializer.  Setting config.reload is
one way to solve this, but another way is to simply clear the cache after engine
initialization.

since engine is marked :after, config has already occured in the Rails.app, so
clearing the cache forces a reload of the correct baseurl.
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.

1 participant