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

Added FuzzyString attribute #97

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
3 participants
@jdufresne

Useful for unique model attributes where the specific value can be fuzzy.

jdufresne added some commits Oct 4, 2013

Added FuzzyString attribute.
Useful for unique model attributes where the specific value can be fuzzy.
@rbarrois

This comment has been minimized.

Show comment
Hide comment
@rbarrois

rbarrois Oct 11, 2013

Member

Sweet!

I think your proposal could be slightly improved by adding an (optional) parameter to set the character set to use.

Could you also add a little bit of documentation?

Thanks :)

Member

rbarrois commented Oct 11, 2013

Sweet!

I think your proposal could be slightly improved by adding an (optional) parameter to set the character set to use.

Could you also add a little bit of documentation?

Thanks :)

Jon Dufresne
Added chars to FuzzyString constructor.
Can now supply any characterset for choosing random characters. Added
documentation.
@jdufresne

This comment has been minimized.

Show comment
Hide comment
@jdufresne

jdufresne Oct 18, 2013

I added a new commit with your suggestions. Please re-review when you have time. Let me know if you would like more documentation or any other changes.

Thanks.

I added a new commit with your suggestions. Please re-review when you have time. Let me know if you would like more documentation or any other changes.

Thanks.

@riyadparvez

This comment has been minimized.

Show comment
Hide comment
@riyadparvez

riyadparvez Oct 23, 2013

This seems to be a great feature. Can you merger this please.

This seems to be a great feature. Can you merger this please.

@rbarrois rbarrois closed this in dc482f6 Oct 28, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment