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

I can't remove any api route #19

Open
github0copilot opened this issue Apr 2, 2024 · 3 comments
Open

I can't remove any api route #19

github0copilot opened this issue Apr 2, 2024 · 3 comments
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers

Comments

@github0copilot
Copy link

image

Upon clicking the delete button, no action is taken. After refreshing the system, it turns white and requires another refresh. Following this, no items are deleted. There are no console.log outputs or error messages, and I am operating in a production build environment.

@Boegie19
Copy link
Member

Boegie19 commented Apr 2, 2024

@github0copilot What version of strapi and of my plugin are you running?

@github0copilot
Copy link
Author

Strapi: 4.21.0
Your Plugin: 1.3.6

I'd like to express my deep appreciation for your hard work. I believe that without your contributions, Strapi would be significantly less effective. Additionally, I've discovered that it's possible to delete and modify things in the development environment, but not in production. This is perfectly acceptable for us, considering our workflow and the existence of pipelines but I am not sure about others.

@Boegie19
Copy link
Member

Boegie19 commented Apr 3, 2024

but not in production.

This is done on purpose, since strapi needs to restart after every change we make, since it adds configuration that can only be changed on startup. since we make changes in the file system. since I don't want protected routes to be saved in the db since now you start a new instance and you lose all your protected routes.

It seems to me like the UI I made is not clear enough that it can't be done in production. I should probably add a banner or remove/gray out the delete button in production.

@Boegie19 Boegie19 added documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers labels Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants