Skip to content

@Stateful - ServerActivityDetailViewModel#1782

Merged
LePips merged 5 commits intojellyfin:mainfrom
JPKribs:statefulServerActivity
Nov 1, 2025
Merged

@Stateful - ServerActivityDetailViewModel#1782
LePips merged 5 commits intojellyfin:mainfrom
JPKribs:statefulServerActivity

Conversation

@JPKribs
Copy link
Member

@JPKribs JPKribs commented Oct 27, 2025

Summary

I was originally planning on doing ServerActivityViewModel as well but then I would need to refactor PagingLibraryViewModel which is a huge undertaking and also appears to be part of: #1752

So, this is a small refactor to the new @Stateful and a TODO in ServerActivityViewModel

@JPKribs JPKribs added the developer Alters the developer experience label Oct 27, 2025
@JPKribs JPKribs requested a review from LePips October 27, 2025 21:35
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.

As a note I changed how libraries work in #1752 and have made the changes to ServerActivityViewModel already.

@LePips LePips merged commit 40b7339 into jellyfin:main Nov 1, 2025
4 checks passed
@JPKribs JPKribs deleted the statefulServerActivity branch November 1, 2025 17:01
@JPKribs JPKribs added iOS Impacts iOS or iPadOS tvOS Impacts tvOS and removed tvOS Impacts tvOS labels Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

developer Alters the developer experience iOS Impacts iOS or iPadOS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants