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

don't use a media Fallback for static assets ( 1461 ) #1480

Merged
merged 1 commit into from
Sep 15, 2024

Conversation

jonashrem
Copy link
Contributor

removes media Fallback in nginx for:

/theme/* /media/* /thumbnail/* /bundles/* /css/* /fonts/* /js/* /recovery/* /sitemap/*

With this, fallback handling, is the same as on caddy

removes media Fallback in nginx for:

/theme/* /media/* /thumbnail/* /bundles/* /css/* /fonts/* /js/* /recovery/* /sitemap/*

With this, fallback handling, is the same as on caddy
@shyim shyim merged commit 51cdba0 into shopware:v6.4 Sep 15, 2024
1 check failed
@shyim
Copy link
Member

shyim commented Sep 15, 2024

Thanks!

@Isengo1989
Copy link
Collaborator

@shyim @jonashrem was this purposly merged into v6.4 only?

The initial one was on 6.6 afaik

image

If it applies also to >= 6.6 please let me know.

@shyim
Copy link
Member

shyim commented Sep 16, 2024

every shopware version :)

@jonashrem
Copy link
Contributor Author

yes, that was a mistake. It can be added to every Shopware version ( or just 6.7 and up if you want to be equivalent to shopware/shopware#4730 )

@jonashrem
Copy link
Contributor Author

@Isengo1989 shall I create a new PR for newer Shopware versions?

@Isengo1989
Copy link
Collaborator

@jonashrem thx, done already

#1482
#1483

6.7 is continuously updated by @bojanrajh, I think -> #1423

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