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

Refactor preference code #1620

Merged
merged 1 commit into from
Apr 15, 2022

Conversation

nielsvanvelzen
Copy link
Member

Some changes to make it easier to work with and maintain the preference code. Blocking function calls removed from the module and replaced with a utility function in the app (only needed for Java compat right now).

Changes

  • Allow migrations in all store types
  • Remove blocking functions from the preference module
  • Move Preference.xx functions to global functions

Issues

Depends on ##1619

@jellyfin-bot jellyfin-bot added the merge conflict Conflicts prevent merging label Apr 13, 2022
@nielsvanvelzen nielsvanvelzen marked this pull request as ready for review April 13, 2022 18:22
@jellyfin-bot jellyfin-bot removed the merge conflict Conflicts prevent merging label Apr 13, 2022
- Allow migrations in all store types
- Remove blocking functions from the preference module
- Move Preference.xx functions to global functions
@nielsvanvelzen nielsvanvelzen changed the title Preference refactor Refactor preference code Apr 15, 2022
@nielsvanvelzen nielsvanvelzen merged commit 33005e3 into jellyfin:master Apr 15, 2022
@nielsvanvelzen nielsvanvelzen deleted the preference-refactor branch April 15, 2022 14:07
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.

3 participants