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

[iOS] Admin Dashboard - Media Access / Deletion Settings #1333

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

JPKribs
Copy link
Member

@JPKribs JPKribs commented Nov 27, 2024

Summary

Creates a menu to enable/disable access to a media library. Also allows for enable/disable access to deleting from a media library.

Getting very close to the end!

Screenshots

New Button New Button Screenshot
View 1 View 1 Screenshot
View 2 View 2 Screenshot

@JPKribs
Copy link
Member Author

JPKribs commented Nov 27, 2024

I think this is good to go unless there is a good binding extension for this scenario. #1323 has this scenario a few different places as well:

There is a list of potential options that should all be their own toggle that come from a datasource. When selected, they should be added to a [String] but when deselected, they should be removed from the [String].

Also, quirks:

  1. I am getting the media libraries from a .getLibraries call on the ServerUserAdminViewModel. I did this because MediaViewModel filters to only compatible libraries and for this instance I need all types. Should I make this it's own viewModel?
  2. I combined Media Access & Media Deletion into the same 'Media Access' view. Do we want these separated? I've already broken the mold from jellyfin-web by not including deletion into [iOS] Admin Dashboard - User Permissions #1313 and IMO Media Access & Media Deletion belong in the same spot but I'm game to change it if it makes more sense elsewhere.

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.

1 participant