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

Fix new Sitemap #2262

Merged
merged 3 commits into from
Mar 18, 2022
Merged

Fix new Sitemap #2262

merged 3 commits into from
Mar 18, 2022

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Mar 14, 2022

What is this pull request for?

Use our ajax lib over browsers fetch.

Our ajax lib takes care of sending the CSRF token and now also works with old Webservers that do not support PATCH HTTP verb.

Notable changes

Webservers do not all understand the PATCH, and PUT HTTP verbs.

Nginx must be compiled with support for it.

Since we cannot know if the web server has been configured as such,
we need to do what Rails does: Send a _method override.

Screenshots

Remove if no visual changes have been made.

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

Webservers do not all understand the PATCH, and PUT HTTP verbs.
Nginx must be compiled with support for it.
Since we cannot know if the web server has been configured as such,
we need to do what Rails does: Send a _method override.
Our ajax lib takes care of sending the CSRF token and works with old Webservers
that do not support PATCH HTTP verb.
@tvdeyen tvdeyen merged commit f745dd0 into AlchemyCMS:main Mar 18, 2022
@tvdeyen tvdeyen deleted the fix-sitemap branch March 18, 2022 11:40
tvdeyen added a commit that referenced this pull request Mar 18, 2022
Backport pull request #2262 from tvdeyen/fix-sitemap
v-ruiz added a commit to v-ruiz/alchemy_cms that referenced this pull request Mar 21, 2022
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.

2 participants