-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Architecture: Interaction with non-dot org plugins as dependencies #4
Comments
I don't think we should disallow installing from 3rd-parties, but we should not add any extra filters etc to accommodate that scenario. |
Seems like it is okay to work the one way but maybe not the other. In other words if I write a premium plugin (paid not on the repo) I could require a .org plugin...but maybe not the other way around. That could lead to some tricky stuff. I think that is an ethical slippery slope that could lead to 'spammy' behavior on requiring plugins.
This kind of raises another important consideration. Not sure what I think about that. What do you guys think? |
@megphillips91 I should work both ways. Requiring a premium plugin, or any non-dot org hosted plugin, will come with a few caveats.
Take a closer look at the above image. It is my version of how dependencies would be organized. A separate tab/view would be available in the I don't really want to get into specifics as my interpretation of how this should be coded may not be what the final version looks like but it certainly is possible. I have actually created a rather simple solution that would require a few other plugins, some premium, but it creates a rather low overhead solution. |
If a non dot org dependency has a valid slug in the In the above, Git Updater has API data from an outside source and |
Adding this as it came up often in the discussion on the Make Post
While this is probably out of scope, I can say that just using my PR it is possible to to include plugins not hosted in the WordPress.org repository simply using existing core filters and adding the slug in the
Requires Plugins
header.The text was updated successfully, but these errors were encountered: