-
Notifications
You must be signed in to change notification settings - Fork 6
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
Recognize plugins #8
Conversation
I really like this idea... I'm struggling a bit with how to do this... "nicely". We should probably have a database online that has this data where people can register their options prefixes. |
I think as a first iteration, it will make it easier to just have it in the plugin, and let people do PR's to add new ones. A bit like with Wappalyzer: https://github.com/HTTPArchive/wappalyzer/tree/main/src/technologies. They do it in json, but I don't think that has added value in this case. But it might be useful to split it in alphabetically separated files, like it's done there. I think maintaining an online database and handling submissions will be a lot of continuous work, and might take a lot of resources later on (although using S3 with downloadable jsons might limit that). You'd also need to pull that in to the plugin. So I would personally prefer the "in the plugin" approach to keep it simple and straight forward. Because the current setup is very simple, this makes it easy to add plugins/prefixes for all contributors, but it is quite possible there is a more efficient or elegant way to do this. On the other hand, I always like to have a starting point, and take it from there. My guess is there will be suggestions for improvements once it's there. Another advantage of keeping it in the plugin is that all dev decisions kan be changed or reverted later on without breaking stuff. |
I just noticed you changed it to json, probably a better standard which is easier to exchange and maintain in the long run. My guess would be that a json will be slightly slower when compared to loading a php file, but in this case performance is probably not an issue, as it only is loaded sporadically on the back-end. |
I was running into more issues as I was playing with this so fixed a few other things along the way 😅 The JSON is easier to maintain I think, and we could read that from a remote thing if we wanted to later on. Going to merge and release :) |
Great! 😀 |
Context
I think it would be really helpful for users if the actual plugin or theme that sets the option is listed. The user can then easily see if the plugin is still installed or not. A later improvement could be to automatically check the active/installed status, and show that as well.
Summary
This PR can be summarized in the following changelog entry:
Relevant technical choices:
Test instructions
Relevant test scenarios
UI changes
Documentation
Quality assurance
Fixes #