-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add status screens for order venues status screens #578
Conversation
…emplates for displaying shift information on a screen in the canteens
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks fine, will have to see it live to say something about the styling
website/status_screen/templates/status_screen/venue_music_screen.html
Outdated
Show resolved
Hide resolved
Hey Vince! Thanks for the MR. I have two suggestions:
I think this is cleaner than refreshing the page (I also got into an infinite refresh loop when testing). |
OAuth2 support for status screen
6e503ff
to
964b301
Compare
@vincevd1 I made some small changes again. I removed the dedicated active shift endpoint in favor of using the list shift endpoint with query parameters |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nicely done @vincevd1! Just one small thing and we can merge this 😄 You may also overwrite the old status screen if you want to.
22cd211
to
b57f5a7
Compare
This adds a status screen that automatically switches between the original status screen for shifts and a music screen to see what is currently playing.
From what I have seen you do not get logged out of the current session as long as the browser stays open which is the case with thaliedje. So we can just log in with an account with no privileges and put this link on.
I hooked up a monitor in noord to show this screen.