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.

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

Send email/text when the user is the nth in line (n chosen by users) #149

Open
Zackhardtoname opened this issue Feb 20, 2020 · 9 comments
Open

Comments

@Zackhardtoname
Copy link

So that the users don't have to constantly check for their spot.
A quick temporary solution might be to call the JS alert function to make an alert?

@mterwill
Copy link
Owner

👋 there is already a feature that will send a browser push notification when you're next up. Would that serve your needs?

A panel like this should show up on the queue page if you've not already dismissed it:

image

@Zackhardtoname
Copy link
Author

Thank you for pointing that out. I actually disabled all notifications by default in Chrome (Version 80.0.3987.132) so I don't think I got the panel (or I missed it...).
Anyhow, after enabling it for the site, it worked well.

I still think there's value in text/email notifications just in case the user has to leave the laptop to find a charging outlet or go to the bathroom.

@Prajna-Ty
Copy link

I agree. That feature would be very helpful. I went to refill my waterbottle yesterday, and by the time when I was back, I was no longer on the queue... :(

@lmcicat
Copy link

lmcicat commented Mar 28, 2020

Hey @mterwill ! @brianguo98 and I need to make open source contributions for our final homework assignment for a class this semester. We were hoping to try to tackle this issue if that is ok with you?

@mterwill
Copy link
Owner

👋 Hey @lmcicat and @brianguo98, that's awesome! Happy & appreciative to have y'alls contribution.

Before you get started, would you mind dropping an outline of how you anticipate the technical implementation of this feature will look and the various components involved?

@lmcicat
Copy link

lmcicat commented Mar 31, 2020

Hey @mterwill , sorry for the delay in response. We unfortunately weren't approved by course staff to work on this repo :(. Once the semester ends, I may come back here and try to tackle this when I have more free time!

@mterwill
Copy link
Owner

mterwill commented Apr 1, 2020

Bummer! Thanks for the heads up.

@annieli555
Copy link
Contributor

Hi again! @pl98 and I are also interested in working on this feature request. Is this request still standing?

@mterwill
Copy link
Owner

mterwill commented Nov 7, 2020

I think this one will be tricky, requiring some infrastructure work as well. Same request as above, if you could outline what you’re thinking and drop it here before starting on any implementation, that’d be great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants