-
Notifications
You must be signed in to change notification settings - Fork 47
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
WP REST API Transport Layer #116
Comments
@emrikol Hi. I'm Tim Dunham from USA Today. I work with Ephraim Gregor. He suggested I work on this issue. I see that a REST API transport layer exists for the old version of the REST API. Are we going to appropriate that file and rewrite it to use the "new" REST API, or is there some reason to keep the existing one and create a different file? Sorry to ask such a simple question, but I'd rather not start off by messing things up. |
Hi @timdunhamusat! The "old" REST API functionality was specifically built for the WordPress.com REST API. Our plan moving forward is to use the core WordPress (.org) REST API. I've unfortunately done a lot of looking to see how much of the current code could be rewritten to use the core REST API, but if it could be reused it would be a great time saver :) Hopefully that answered your question. If not, or if you have others, please let me know. Thanks! |
Hi @emrikol and @philipjohn, I'm about to start working on this. I have two initial questions:
Thanks! |
We also need to make sure that the existing REST PUSH client works with WordPress.com first.
For any WP REST API endpoints let's stick to OAuth for now. Also cc @joshbetz for any thoughts. |
The primary transport layer out of the box should be WP REST API.
This would also include implementing any necessary authentication support for a "push" transport.
The text was updated successfully, but these errors were encountered: