-
Notifications
You must be signed in to change notification settings - Fork 116
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
Introduce the extensions screen #396
Comments
"Ability to install (download) an extension from this screen (is this even possible?)" - I don't see why not :) For the three "connect to wp-stream.com" items... what do we think about using OAuth for this? Pros? Cons? One pro I can think of is that it's a pre-existing standard for this type of relationship. But, would it be more or less work than what is currently proposed? |
@Japh Actually I think @shadyvb already has the authentication part working, although I'm not sure about the method. Care to comment, Shady? @c3mdigital I just want the whole experience to be clean, easy and dead simple. Right now I think it's a little buggy and not totally completed, so it will be up to you to improve this process and test all the scenarios a user might perform. |
- Add admin_menu callback to admin - Added require_once for extension file to WP_Stream - Created base WP_Stream_Extension class - Skeleton class methods to handle displaying, activating, downloading, authorizing premium extensions - Should update routine #400 be initiated here?
@c3mdigital So is the intention here for the new |
@fjarrett Yes, I'm going to copy over the code for accessing the api and outputting the extensions on the screen. The existing template code is just the backbone template from the themes page for reference. |
- Update branch with code from extensions branch - Organize page output into separate methods to each perform specific tasks - Added singleton get_instance method for initializing the class
@fjarrett Do we know where we are going to store the zipped extension so it can be installed from the extension screen? For my testing I put it in |
@c3mdigital I don't know but @shadyvb does. Shady any advice? |
@c3mdigital Zips will be stored outside of the docroot, to prevent any unauthorized access. They're typically stored in |
@Japh The existing implementation in |
@shadyvb yes for sure. |
@c3mdigital I just added another requirement for Multisite support. |
@c3mdigital Let me know when you're ready for me to merge this into the issue-65 branch so I can test it on multisite. (see #401). |
@fjarrett @c3mdigital Just checked the three tasks about the license activation. |
This screen will introduce users to the optional extensions that are available as part of Stream Premium, as well as authorize the site to receive auto-updates.
There has been some progress made already in the
extensions
branch.Note: You will need to have vvv.wp-stream.com setup locally which will house all of the extension data and expose the data using the JSON REST API plugin.
Preview
The text was updated successfully, but these errors were encountered: