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

[WIP] [iOS] Admin Dashboard Enhancements #1271

Closed
wants to merge 14 commits into from

Conversation

JPKribs
Copy link
Member

@JPKribs JPKribs commented Oct 14, 2024

Formerly

#1255

Issue

This is an extension of #1230

Summary

This PR is where I have a couple different branches living together. I need to split them out into more manageable PRs but I wanted to make this visible so it wasn't just me working in the shadows. All of the Goals below are in this branch with varying levels of polish. As items are done, I'll check them off below.

Goals

ActiveSessionsIndicator

4BF72DC7-FD87-4CF7-B04E-67CD8386840E_4_5005_c

JPKribs and others added 14 commits October 7, 2024 15:58
…. Remove leftover EditTaskTriggerView logic. Add Results/Errors to the EditScheduledTaskView. Track Progress on EditScheduledTaskView.
… struct. Creation of a ChevronInputButton object that probably needs some love...
… BEFORE attempting to delete. Creation of an Admin Session Indicator for showing if there are active sessions from the HomeView. Fix Day of Week localization based on jellyfin#1255 (comment) - Confirmed working! Created an activeSessions published value for usage with the Active Sessions Indicator. Migration of the ChevronInputButton to allow more input types than just TextFields. Creation of a Users section. Grid vs Row User Section. Allow for User Password Resets. Make sure the DeviceType backgrounds never clash with the SVG. General cleanup of previous work mostly labels and formatting based on changes mentioned above.

Finally, migrated ALL of the UserDashboard -> AdminDashboard (Since it's admin only now) and created it's own coordinator. Now available from the ActiveSessionsIndicator.
@JPKribs JPKribs changed the title Admin dashboard enhancements [iOS] Admin dashboard enhancements Oct 14, 2024
@JPKribs JPKribs changed the title [iOS] Admin dashboard enhancements [iOS] Admin Dashboard Enhancements Oct 14, 2024
Copy link
Member

@LePips LePips left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was more digestible when it was mainly task triggers and small work to devices, but now is getting too large for me to review effectively and make comments that apply to future work.

Apologies, butI ask that all new sections be implemented in their own PRs as they are quite independent of one another (sans xcodeproj conflicts). This would make my reviews of each section faster and more effective.

@JPKribs
Copy link
Member Author

JPKribs commented Oct 14, 2024

This was more digestible when it was mainly task triggers and small work to devices, but now is getting too large for me to review effectively and make comments that apply to future work.

Apologies, butI ask that all new sections be implemented in their own PRs as they are quite independent of one another (sans xcodeproj conflicts). This would make my reviews of each section faster and more effective.

Sounds good. For now, a lot of the work builds on other items so I'm just getting this into a single branch. When I'm ready, I'm going to break this out into something more reviewable by section.

@JPKribs
Copy link
Member Author

JPKribs commented Oct 14, 2024

Apologies for that! The primary goal for this branch was to just visibility. But, I completely agree. This is kind of a beast to look at.

@JPKribs JPKribs changed the title [iOS] Admin Dashboard Enhancements [WIP] [iOS] Admin Dashboard Enhancements Oct 16, 2024
@JPKribs
Copy link
Member Author

JPKribs commented Oct 20, 2024

I'm closing this out in favor of smaller, actually manageable PRs. Tracking for this will occur:

#1283

@JPKribs JPKribs closed this Oct 20, 2024
@JPKribs JPKribs deleted the adminDashboardEnhancements branch October 20, 2024 05:13
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