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

Email validation #29

Closed
naixx opened this issue Apr 7, 2014 · 3 comments
Closed

Email validation #29

naixx opened this issue Apr 7, 2014 · 3 comments

Comments

@naixx
Copy link

naixx commented Apr 7, 2014

First of all, this is a great library!

But I have a problem validating email with + sign. I see, that a fix is already in repo, but not in the latest release.

Do you plan any minor release to maven?

@ragunathjawahar
Copy link
Owner

Due to the lack of tests now I'm unable to verify if it satisfies your requirement. The email RegEx still seems to have some issues. Could you please post the email that you want to check? After verifying I could do a minor release to MC.

@naixx
Copy link
Author

naixx commented Apr 10, 2014

@ragunathjawahar simply add + sign. I think, using RegEx from android.util.Patterns is enough tested for this. Just s you did in one of latest commits.

@ragunathjawahar
Copy link
Owner

@naixx Staged the release, should be available in a few hours. Please let me know if this fixes the issue.

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

No branches or pull requests

2 participants