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

Conversation

@joehoyle
Copy link
Member

No description provided.

Copy link
Member

Choose a reason for hiding this comment

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

@joehoyle can this just use assertFalse()?

Copy link
Member Author

Choose a reason for hiding this comment

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

I could - given that I was comparing the equality in a few fields - it kind of meant sense to keep it consistent with that.

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed in 5ac10a6

joehoyle added a commit that referenced this pull request Jun 12, 2015
This default is then use for the default arg value also. We could use
this in #1345 however, we want the default specifically to be public
so we can communicate to clients what th default value will be.
joehoyle and others added 3 commits June 11, 2015 18:26
# Conflicts:
#	tests/class-wp-rest-test-controller.php
#	tests/test-rest-controller.php
rachelbaker added a commit that referenced this pull request Jun 12, 2015
Ability to specify `arg_options` in the schema to override the arguments
@rachelbaker rachelbaker merged commit ad38f06 into develop Jun 12, 2015
@rachelbaker rachelbaker deleted the schema-arg-options branch June 12, 2015 14:22
@rachelbaker
Copy link
Member

Merged #1345

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.

3 participants