Skip to content
This repository has been archived by the owner on Jun 7, 2020. It is now read-only.

Error message after adding new employee #18

Open
yoowul opened this issue May 4, 2017 · 2 comments
Open

Error message after adding new employee #18

yoowul opened this issue May 4, 2017 · 2 comments

Comments

@yoowul
Copy link
Contributor

yoowul commented May 4, 2017

After creating an employee with an unused email the app throws an error indicating that the emal is used, even though the account has been created anyway.

@yoowul yoowul added the bug label May 4, 2017
@wyspiansky wyspiansky added major and removed bug labels May 4, 2017
@yoowul
Copy link
Contributor Author

yoowul commented May 9, 2017

Cannot reproduce after switching mail service. Error message may be due to error handler misinterpreting e-mail sending failure as violation of unique e-mail constraint.

@yoowul yoowul added minor and removed major labels May 9, 2017
@cameel
Copy link
Member

cameel commented May 9, 2017

You need to fix the overly broad exception handlers.

@yoowul yoowul added the bug label May 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants