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

add forbiddenKeys method to object #1183

Merged
merged 4 commits into from
May 13, 2017
Merged

add forbiddenKeys method to object #1183

merged 4 commits into from
May 13, 2017

Conversation

danielo515
Copy link
Contributor

This will allow very interesting modular systems where you can define a global very generic object where all keys are optional. Then each module can add their own restrictions using that base object, allowing, disallowing or making required certain properties. Combine this with some default values and you will have a very strong and modular system with high consistency on the results.

This will allow very interesting modular systems where you can define a global very generic object. Then each module can add their own restrictions using that base object, allowing, disallowing or making required certain properties. Combine this with some default values and you will have a very strong and modular system with high consistency on the results.
@Marsup Marsup added the feature New functionality or improvement label May 13, 2017
@Marsup Marsup self-assigned this May 13, 2017
@Marsup Marsup added this to the 10.5.0 milestone May 13, 2017
@Marsup
Copy link
Collaborator

Marsup commented May 13, 2017

I have nothing against this change, but you seem a little too excited about such a small change, it's not a revolution :D

@Marsup Marsup merged commit dcc6f4c into hapijs:master May 13, 2017
Marsup added a commit that referenced this pull request May 13, 2017
@danielo515
Copy link
Contributor Author

@Marsup depends on​ how much you narrow your point of view 😄
However,since this kid of extensions is not possible using the current extensions mechanism it can be considered some kind of revolution to someone that depends on it (someone like me!)

Thank you very much. Sorry about my excitement but I'm having a lot of fun with joi recently .

Regards

@danielo515
Copy link
Contributor Author

@Marsup thanks for merging and documenting this.
When is the release of 10.5.0 planned ? Can I start using it with the @10.5.0 tag?

@Marsup
Copy link
Collaborator

Marsup commented May 18, 2017

Released.

@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
feature New functionality or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants