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

Renaissance (partial rewrite) #98

Merged
merged 30 commits into from
Dec 7, 2015
Merged

Renaissance (partial rewrite) #98

merged 30 commits into from
Dec 7, 2015

Conversation

rmccue
Copy link
Member

@rmccue rmccue commented Nov 23, 2015

There are tonnes of open issues here, and they need fixing. The best way to do this is hunkering down and doing them all at once by going back to the drawing board.

This fixes a tonne of the outstanding issues and adds a bit of polish. It's not done yet, but it's forward progress.

@rmccue rmccue added this to the 1.0 milestone Nov 23, 2015
@rmccue
Copy link
Member Author

rmccue commented Nov 23, 2015

Note: this will partially invalidate existing clients, as they now need a callback saved in the database to be used as the base for checking against. Already-issued access tokens will continue to work, but you won't be able to create new request tokens or authorize them.

@kosso
Copy link
Contributor

kosso commented Nov 23, 2015

👍 Thanks. This works for me. I had to create a new consumer to get one of the old ones working again though.

btw: Not too sure how to implement your workaround for #95 using WP_REST_Server in get_parameters().

@rmccue
Copy link
Member Author

rmccue commented Dec 7, 2015

It's go time!

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants