-
Notifications
You must be signed in to change notification settings - Fork 0
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 the option to shuffle playlists and swap the order of your songs. #139
Conversation
…en videos of the same user)
I have now tested this with an admin user and a non admin user at the same time and the permissions work as expected. |
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.
server/.env.example
Outdated
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.
This doesnt need to be vite right?
} | ||
if (up) { |
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.
} | |
if (up) { | |
} else { |
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.
Nice typescript!
}; | ||
|
||
const maxVideos = parseInt( | ||
import.meta.env.VITE_USER_MAX_VIDEOS_IN_QUEUE ?? "20" |
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.
Deze importeer je toch via een .env in de client map? En niet via de .env in de server map?
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.
nee vite fixt dit automatisch voor je
When adding a playlist you now get a modal that lets you choose if you want the playlist to be shuffled or not (see picture). It also informs non admins that only the first x songs get added. Fixes saproto/saproto#2218.
Also adds the option to swap videos that you have put in from position, and lets admins do that for other users.
This means if you have queue position 3 and 5 under the fair add system, you can press the up arrow on the song in position 3 or the down arrow in position 5 and than these two items will be swapped in the queue. See picture for the UI.