Found a small bug (race condition?) @sam. Two different users can assign the topic at the same time.
Not much we can do here sadly, we allow already assigned topics to be assigned to somebody new.
This is not a new person though it is the same person.
Ahh did not notice that, I guess we can error out here.
- IF topic is already assigned
- AND you are attempting to assign to the same person already assigned
- RAISE A non 400 error “Topic is already assigned to XYZ” and display to client.
- Test on both mobile and desktop
- Include a controller test at least in the PR
I will try to submit PR for this.
This is fixed now.
This topic was automatically closed after 34 hours. New replies are no longer allowed.