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

Update nginx.conf #743

Closed
wants to merge 1 commit into from
Closed

Update nginx.conf #743

wants to merge 1 commit into from

Conversation

mrhein
Copy link
Contributor

@mrhein mrhein commented Mar 23, 2016

@rhukster
Copy link
Member

Looks good. I will test when I get home from vacation this weekend. Thanks!

@rondlite
Copy link

rondlite commented Apr 4, 2016

the .js javascript header found in example config should be editted too. Otherwise all .json uri's are parsed as files and you can not clear cache of preview your work in the admin panel.

@rhukster
Copy link
Member

rhukster commented Apr 4, 2016

@rondlite can you provide a code snipped with .js fix?

@rondlite
Copy link

rondlite commented Apr 4, 2016

@rhukster soz should have posted with previous post
location ~ \.js$ { add_header Content-Type application/x-javascript; }
just added EOL after the .js

also, not sure if is needed but my nginx.conf I made like this (from the caddy example):

location / { try_files $uri $uri/ /index.php?_url=$uri; }
added URI to index

@rhukster
Copy link
Member

rhukster commented Jun 3, 2016

i've added the updated to try_files manually.

@rhukster rhukster closed this Jun 3, 2016
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.

3 participants