Skip to content

Update Nginx .conf to fix .php editing issues#126

Closed
PalmarHealer wants to merge 1 commit intopelican-dev:mainfrom
PalmarHealer:patch-1
Closed

Update Nginx .conf to fix .php editing issues#126
PalmarHealer wants to merge 1 commit intopelican-dev:mainfrom
PalmarHealer:patch-1

Conversation

@PalmarHealer
Copy link

Change Nginx config to specifically route to /index.php if uri contains a /edit/. This fixes:
pelican-dev/panel#746
pelican-dev/panel#1150
And propably this one aswell:
pelican-dev/panel#1105

Update to fix .php editing issues
@PalmarHealer PalmarHealer requested a review from a team as a code owner March 22, 2025 16:43
@netlify
Copy link

netlify bot commented Mar 22, 2025

Deploy Preview for pelica ready!

Name Link
🔨 Latest commit a0b366b
🔍 Latest deploy log https://app.netlify.com/sites/pelica/deploys/67dee8b695b6910008e3c079
😎 Deploy Preview https://deploy-preview-126--pelica.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@rmartinoscar
Copy link
Member

This would only "fix" it for Nginx but not for other webservers.
So its a temp fix

@PalmarHealer
Copy link
Author

So then either all of the web server configs are adjusted or the way how the URIs are being build has to be changed. Like replacing dots with a placeholder. But I don't know how to change that...

Personally I would go the config way but that could cause compatibility issues in the future.

@PalmarHealer PalmarHealer marked this pull request as draft March 22, 2025 20:15
@Boy132
Copy link
Member

Boy132 commented Mar 31, 2025

Thanks, but this should be fixed on the panel side and not via the webserver config.

@Boy132 Boy132 closed this Mar 31, 2025
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