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

Remove validate email #442

Merged
merged 3 commits into from
Aug 26, 2019
Merged

Conversation

PrimozGodec
Copy link
Collaborator

Issue

Fixes #440

Description of changes

With this pull request we:

  • remove valiate_email dependency from the add-on - it is now checked with regex in the widget
  • save email through CredentialManager
Includes
  • Code changes
  • Tests
  • Documentation

@codecov-io
Copy link

codecov-io commented Jun 20, 2019

Codecov Report

Merging #442 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #442      +/-   ##
==========================================
- Coverage   85.59%   85.57%   -0.03%     
==========================================
  Files          34       34              
  Lines        1937     1934       -3     
  Branches      331      330       -1     
==========================================
- Hits         1658     1655       -3     
  Misses        239      239              
  Partials       40       40

@PrimozGodec PrimozGodec force-pushed the remove-validate_email branch 2 times, most recently from 486497b to 07de368 Compare August 23, 2019 14:17
@ajdapretnar
Copy link
Collaborator

A comment here. When opening the email window, a warning should appear after the users types something and presses enter. I don't like being greeted by an error when I haven't even done anything. Also, I think it would be nice to have focus on the text line.

@PrimozGodec PrimozGodec force-pushed the remove-validate_email branch from 07de368 to 075761b Compare August 26, 2019 10:22
@PrimozGodec
Copy link
Collaborator Author

@ajdapretnar I agree with you. I have addressed the issues. :)

@PrimozGodec PrimozGodec force-pushed the remove-validate_email branch from 075761b to fd0c256 Compare August 26, 2019 12:24
@PrimozGodec
Copy link
Collaborator Author

@ajdapretnar tests are passing now. :)

@ajdapretnar ajdapretnar merged commit 3631313 into biolab:master Aug 26, 2019
@PrimozGodec PrimozGodec deleted the remove-validate_email branch March 29, 2023 10:38
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.

Remove validate_email
3 participants