Skip to content

Bug Hunt: object.and is supposed to be all-or-nothing#284

Merged
hueniverse merged 1 commit intohapijs:masterfrom
iceddev:and
Apr 23, 2014
Merged

Bug Hunt: object.and is supposed to be all-or-nothing#284
hueniverse merged 1 commit intohapijs:masterfrom
iceddev:and

Conversation

@phated
Copy link
Contributor

@phated phated commented Apr 23, 2014

If all of the keys are missing, the and rule should pass. This avoids having to do .with('a', 'b').with('b', 'a').

phated added a commit to iceddev/joi that referenced this pull request Apr 23, 2014
hueniverse pushed a commit that referenced this pull request Apr 23, 2014
Bug Hunt: object.and is supposed to be all-or-nothing
@hueniverse hueniverse merged commit 99e6a53 into hapijs:master Apr 23, 2014
hueniverse pushed a commit that referenced this pull request Apr 23, 2014
Fix to make object.and all-or-nothing - fixes #284
@hueniverse hueniverse added this to the 4.1.0 milestone Apr 23, 2014
@hueniverse hueniverse self-assigned this Apr 23, 2014
@hueniverse
Copy link
Contributor

Confirmed! $50

@hueniverse
Copy link
Contributor

@phated please contact me at eran@hammer.io to arrange award delivery. Thanks!

@phated
Copy link
Contributor Author

phated commented Jun 4, 2014

@hueniverse did you receive my email?

@hueniverse
Copy link
Contributor

@phated yes, sorry. I'm behind.

@phated
Copy link
Contributor Author

phated commented Jun 4, 2014

@hueniverse np, just wanted to confirm. Thanks.

@lock
Copy link

lock bot commented Jan 9, 2020

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 9, 2020
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.

2 participants