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

Use path instead of url, no work with no request. #1193

Merged
merged 1 commit into from Mar 8, 2017

Conversation

Projects
None yet
2 participants
@merqlove
Contributor

merqlove commented Feb 2, 2017

No description provided.

@did

This comment has been minimized.

Show comment
Hide comment
@did

did Mar 8, 2017

Member

hey @merqlove, I understand your patch but I don't get how it could happen. Is there a case where we'd have to render the "wysihtml_css.rb" file without a request?

Member

did commented Mar 8, 2017

hey @merqlove, I understand your patch but I don't get how it could happen. Is there a case where we'd have to render the "wysihtml_css.rb" file without a request?

@merqlove

This comment has been minimized.

Show comment
Hide comment
@merqlove

merqlove Mar 8, 2017

Contributor

It is happen when you have new site without content (for example you have changed language).

Contributor

merqlove commented Mar 8, 2017

It is happen when you have new site without content (for example you have changed language).

@did

This comment has been minimized.

Show comment
Hide comment
@did

did Mar 8, 2017

Member

hmmm I think that's because of your main app settings but still, stylesheet_path is probably better here (than stylesheet_url).

Member

did commented Mar 8, 2017

hmmm I think that's because of your main app settings but still, stylesheet_path is probably better here (than stylesheet_url).

@did did merged commit 2af85a2 into locomotivecms:master Mar 8, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment