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

Boolean Validator uses <code> instead of ' #502

Merged
merged 1 commit into from
Jan 30, 2017
Merged

Boolean Validator uses <code> instead of ' #502

merged 1 commit into from
Jan 30, 2017

Conversation

berfarah
Copy link
Contributor

@berfarah berfarah commented Nov 5, 2016

Updated the Boolean Validator to output its message similarly to when you have array validations

This:

param :value, :bool

image

Will now look like this:

param :value, [true, false, 0, 1]

image

when reading the documentation. This makes it clearer that you can pass in the literal values instead of strings of those values.

I can update the ArrayValidator to do the same thing –– I believe it's currently just the Enum one that outputs it with a code.

Updated the Boolean Validator to output its message similarly to when you have array validations
@iNecas
Copy link
Member

iNecas commented Jan 30, 2017

Thanks @berfarah , Merging now

@iNecas iNecas merged commit 16144d7 into Apipie:master Jan 30, 2017
@iNecas
Copy link
Member

iNecas commented Feb 18, 2017

apipie-rails-0.4.0 has just been released, including this enhancement. Thanks @berfarah for help!

@berfarah
Copy link
Contributor Author

Happy to contribute, even if it was a tiny bit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants