[Meta] Move Defaults to StoredValues #1399
Replies: 1 comment
-
@LePips Please do NOT consider this a priority in any way! I'm going to be a lot slower as I enter a busy season at my real job but I wanted to throw this out there and get a discussion going before I put in time doing this. I think this work be a good value add cleaning this up and, frankly, this is a place that doesn't require a lot of logic or thinking. Instead, it's more of just... find & replace. So, I thought this would be a good project, IF we feel it would be beneficial! Let me know what you think! I personally like the structure in the code up above but I'm not really a developer by trade so I want to lean on your expertise here. I think something like:
Is good and descriptive but I could also absolutely see the argument of this being overly long and verbose. When you have a free chance, let me know what you think! |
Beta Was this translation helpful? Give feedback.
-
I have a bit of time on my hands for more grunt-type work and I thought I would see if we wanted to start the move over from Defaults to StoredValues. In my mind, it makes sense to get this all sorted out prior to a tvOS release.
My thinking is that iOS you're not very likely to have different user profiles as most iOS and iPadOS devices are used by a single person. Therefore, the shared Default settings aren't a major issue. With tvOS, multiple users will exist so the headache of shared settings will be more prevalent.
As I understand it, there are three steps to this process:
I am going to add an unofficial fourth & fifth step in here
See:
Here is my initial attempt. I wanted to make this discussion to see if my format makes sense before I proceeded. Below is my code but here is the branch I did this initial setup on:
https://github.com/JPKribs/Swiftfin/tree/defaultsToUserSettings
Beta Was this translation helpful? Give feedback.
All reactions