Skip to content
This repository was archived by the owner on Sep 24, 2018. It is now read-only.

Conversation

@adamsilverstein
Copy link

  • Initial pass at bringing the library up to date
  • Correct model endpoints that have changed
  • Verified all models read correctly
  • Verified Post create, read, destroy (trash)
  • Clean up doc block language for models
  • Add additional JSDOC3 Doc Blocks for models
  • rename internal BaseModel to WPApiBaseModel
  • user model defaults to 'me' route if no user id is passed
  • remove slug default from Post model, passing a blank slug was causing Create to fail.
  • Consider eliminating defauls, the api already has defaults?
  • Correct overriding of 'delete', backbone model funciton is destroy

* Initial pass at bringing the library up to date
* Correct model endpoints that have changed
* Verified all models read correctly
* Verified Post create, read, destroy (trash)
* Clean up doc block language for models
* Add additional JSDOC3 Doc Blocks for models
* rename internal BaseModel to WPApiBaseModel
* user model defaults to 'me' route if no user id is passed
* remove slug default from Post model, passing a blank slug was causing Create to fail.
* Consider eliminating defauls, the api already has defaults?
* Correct overriding of 'delete', backbone model funciton is `destroy`
@danielbachhuber danielbachhuber added this to the 2.0 Beta 9 milestone Dec 10, 2015
@danielbachhuber
Copy link
Member

🚢 🚢

danielbachhuber added a commit that referenced this pull request Dec 10, 2015
Bring the wp-api.js library up to date for v2 of the REST API
@danielbachhuber danielbachhuber merged commit eac563b into develop Dec 10, 2015
@danielbachhuber danielbachhuber deleted the fixes/update-wp-api-js-library branch December 10, 2015 14:54
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will now need updated per #1802

@adamsilverstein
Copy link
Author

@joehoyle the js client now traverses the schema to determine models to build, so no update will be required :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants