Skip to content

playlist user controls#994

Open
ajohn723 wants to merge 47 commits into
mainfrom
feature/playlist-user-operations
Open

playlist user controls#994
ajohn723 wants to merge 47 commits into
mainfrom
feature/playlist-user-operations

Conversation

@ajohn723
Copy link
Copy Markdown
Collaborator

@ajohn723 ajohn723 commented Feb 13, 2026

What is the change

collaborative playlist feature

What does this address

Issue number / link

Tag reviewers

@anultravioletaurora

Comment thread src/api/queries/playlist/utils/users.ts Outdated
Comment thread src/api/queries/playlist/utils/users.ts Outdated
Comment thread src/api/queries/playlist/utils/users.ts Outdated
Comment thread src/api/queries/playlist/utils/users.ts Outdated
Comment thread src/api/queries/playlist/utils/users.ts Outdated
@anultravioletaurora
Copy link
Copy Markdown
Member

Now that I think about it - we might need another hook to retrieve the list of users.

We'd create a hook - useUsers - that would call a function, fetchUsers

This fetchUsers function would retrieve the list of users from Jellyfin, using a similar format to the other fetchPlaylistUsers function we added

You'll use getUsersApi to get the UsersApi, and then call one of it's functions to fetch the users 👍

@ajohn723
Copy link
Copy Markdown
Collaborator Author

ajohn723 commented Apr 7, 2026

work on establishing playlist ownership (users added to the playlist should not be able to share with others if they do not own it)

@ajohn723
Copy link
Copy Markdown
Collaborator Author

ajohn723 commented Apr 7, 2026

animations - as a user gets added, it animates it into the other list section ('shared with')

@ajohn723
Copy link
Copy Markdown
Collaborator Author

ajohn723 commented Apr 7, 2026

wire up user search to add users

@ajohn723
Copy link
Copy Markdown
Collaborator Author

ajohn723 commented Apr 7, 2026

toast for shared playlists @anultravioletaurora (in relation to playlist ownership perhaps?)

@anultravioletaurora
Copy link
Copy Markdown
Member

anultravioletaurora commented Apr 22, 2026

@ajohn723 I did a pull from main on your branch, you will need to run bun i the next time you pull 👍

EDIT: JK 😆 just do a pull, I added a hook that should fire when you do a git pull that will update your node_modules

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

Successfully merging this pull request may close these issues.

2 participants