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

bug fixes (#12), Gemfile grouping, subdir feature (#13) and css cleanup #47

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

Conversation

stbuehler
Copy link
Contributor

@FooBarWidget
Copy link
Member

Sorry for taking so long to reply. Does this pull request obsolete issue #13?

@FooBarWidget
Copy link
Member

I've commited 2cf73f1 and 4c0641f for now.

@stbuehler
Copy link
Contributor Author

Yes, the commit with (fixes #13) should implement the feature requested in #13 :)

@FooBarWidget
Copy link
Member

It looks like I don't have enough time to get this merged in the short term. I'm preparing a conference trip and I'll be away for a week. I'll have a better look when I'm back.

@stbuehler
Copy link
Contributor Author

Back to work? http://youtu.be/YaQYrVQDPHE?t=6s :)

`config.action_controller.relative_url_root` can be used to tell Rails
that you are deploying to a subdirectory. The default is
`ENV['RAILS_RELATIVE_URL_ROOT']`.

the asset pipeline doesn't like using the value from request script_name,
although this should contain the correct value and should theoretically
be possible :(
…o, remove font for comments

 * the comments should use the same font as the page they are embedded in
@stbuehler
Copy link
Contributor Author

I removed the hack to auto-detect the subdir mount; as i was migration juvia to rails 4 the hack broke, and the internal handling got way too complicated to workaround it again.

This also means that I didn't test it; my previous tests were done on my main juvia instance which was mounted at different urls, but now is only mounted at '/' (you can configure only one static subdir).

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.

Juvia not working properly when deployed to a sub URI Comment.accessible_by broken
2 participants