Implement not allowing multiple queue joins #116

mterwill / office-hours-help-queue

A queue to help manage office hours for large courses

GNU General Public License v3.0

84 stars 21 forks source link

Implement not allowing multiple queue joins #116 #134

Closed MiloATH closed 5 years ago

MiloATH commented 5 years ago

Prevents a student from being on multiple queues for the same course.

Doesn't handle on the frontend too well. Just doesn't submit the request. Better would be provide an error to the user that they are already in another queue.

Alternatively, if they request being on a new queue, we could drop them from all other queues?

MiloATH commented 5 years ago

As mentioned in #102, front end error handling would be great.

MiloATH commented 5 years ago

Build seems to be passing on travis-ci, but it doesn't seem to be populating to github, https://travis-ci.org/mterwill/office-hours-help-queue/builds/522882283.

mterwill commented 5 years ago

Deployed! 🙌 Closes #116