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

Invite existing users gives an IntegrityError #2274

Closed
KasperBrandt opened this issue Jun 28, 2016 · 1 comment
Closed

Invite existing users gives an IntegrityError #2274

KasperBrandt opened this issue Jun 28, 2016 · 1 comment
Assignees
@damienallen
Labels
Bug Type: MyRSR
Milestone
3.15 Suva

Comments

@KasperBrandt
Copy link
Contributor

See http://sentry.support.akvo-ops.org/rsr/live/group/797/.
@KasperBrandt KasperBrandt added this to the 3.15 Suva milestone Jun 28, 2016
@KasperBrandt KasperBrandt self-assigned this Jun 28, 2016
@damienallen
Copy link
Contributor

damienallen commented Jul 4, 2016 •
edited
Loading

Test plan

Add a second employment for a user and attempt to change the group to the same value as an existing employment. An IntegrityError should no longer be raised, and a message below the dropdown should inform the user that the employment group already exists for the selected user. The error message should disappear if a valid group is selected.

damienallen added a commit that referenced this issue Jul 4, 2016
@damienallen
[Fixes #2274] Error is now caught and conveyed to user
c3f2392
damienallen added a commit that referenced this issue Jul 4, 2016
@damienallen
[Fixes #2274] Better wording for error message
2d725f8
damienallen added a commit that referenced this issue Jul 4, 2016
@damienallen
[Fixes #2274] Fixed indentation
4de9d2d
damienallen added a commit that referenced this issue Jul 4, 2016
@damienallen
[Fixes #2274] Code cleanup
8efdf56
@damienallen damienallen mentioned this issue Jul 4, 2016
Merged
KasperBrandt added a commit that referenced this issue Jul 6, 2016
@KasperBrandt
Merge pull request #2290 from akvo/#2274-invite-existing-users
4763c2f 
[#2274] Prevent selection of duplicate employment groups
@KasperBrandt KasperBrandt mentioned this issue Jul 13, 2016
Closed
@KasperBrandt KasperBrandt mentioned this issue Jul 21, 2016
Closed
@MichaelAkvo MichaelAkvo added this to RSR Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@damienallen damienallen

Labels
Bug Type: MyRSR
Projects
RSR
Archived in project
Milestone
3.15 Suva
Development

No branches or pull requests
2 participants
@KasperBrandt @damienallen