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

Conversation

@rachelbaker
Copy link
Member

No description provided.

@rachelbaker
Copy link
Member Author

@WP-API/amigos #reviewmerge

@rachelbaker rachelbaker added this to the 2.0 Beta 6 milestone Nov 1, 2015
@danielbachhuber
Copy link
Member

order of importance

How did you determine this?

@rmccue
Copy link
Member

rmccue commented Nov 2, 2015

👍

@danielbachhuber At a guess, just a design decision.

@rachelbaker
Copy link
Member Author

@danielbachhuber how did you determine the order of the other endpoint schemas?

I moved id to the top and then tried to order the other properties by making educated guesses on likely use.

I can see there are some things I missed here:

  • url should probably move up
  • capabilities and extra_capabilities should be under role and roles

I will refresh this PR when I can.

@danielbachhuber
Copy link
Member

@WP-API/amigos #reviewmerge

I moved url to under email, where it appears in the User Profile in the dashboard. I already see capabilities and extra_capabilities to be under role and roles.

joehoyle added a commit that referenced this pull request Nov 9, 2015
Change users schema order to be order of importance instead of alpha
@joehoyle joehoyle merged commit 292cef8 into develop Nov 9, 2015
@joehoyle joehoyle deleted the reorder-user-schema branch November 9, 2015 23:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants