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.

Sign up for GitHub

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

Jump to bottom

Specify validation error for email validation checks #118

Closed
mbeacom opened this issue Dec 14, 2017 · 0 comments · Fixed by #122
Closed

Specify validation error for email validation checks #118

mbeacom opened this issue Dec 14, 2017 · 0 comments · Fixed by #122

Comments

@mbeacom
Copy link
Contributor

mbeacom commented Dec 14, 2017

Description

Every location that validates email addresses currently is caught by a bare exception.

Possible Implementation

Replace the base except with ValidationError
@mbeacom mbeacom mentioned this issue Dec 14, 2017
Merged
3 tasks
ethikz pushed a commit to ethikz/web that referenced this issue Jan 24, 2018
@mbeacom
Fixes gitcoinco#118 - Use ValidationError over bare exception for ema…
8d62ba0 
…il validation
ethikz pushed a commit to ethikz/web that referenced this issue Jan 24, 2018
@mbeacom
Fixes gitcoinco#118 - Use ValidationError over bare exception for ema…
ac88050 
…il validation
ethikz pushed a commit to ethikz/web that referenced this issue Jan 24, 2018
@mbeacom @ethikz
Fixes gitcoinco#118 - Use ValidationError over bare exception for ema…
07b2bc4 
…il validation
ethikz pushed a commit to ethikz/web that referenced this issue Jan 24, 2018
@mbeacom @ethikz
Fixes gitcoinco#118 - Use ValidationError over bare exception for ema…
145b179 
…il validation
ethikz pushed a commit to ethikz/web that referenced this issue Jan 25, 2018
@mbeacom
Fixes gitcoinco#118 - Use ValidationError over bare exception for ema…
b16b10b 
…il validation
ethikz pushed a commit to ethikz/web that referenced this issue Jan 27, 2018
@mbeacom
Fixes gitcoinco#118 - Use ValidationError over bare exception for ema…
2b3f331 
…il validation
ethikz pushed a commit to ethikz/web that referenced this issue Jan 29, 2018
@mbeacom
Fixes gitcoinco#118 - Use ValidationError over bare exception for ema…
242f97d 
…il validation
ethikz pushed a commit to ethikz/web that referenced this issue Jan 29, 2018
@mbeacom
Fixes gitcoinco#118 - Use ValidationError over bare exception for ema…
a095b7b 
…il validation
@octavioamu octavioamu mentioned this issue Oct 31, 2018
Closed
owocki pushed a commit that referenced this issue Apr 30, 2021
@scco
Merge pull request #118 from nopslip/QL2_richard_frontend
ef405af 
update on dashboard
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

General python cleanup
1 participant
@mbeacom