ReVanced / revanced-manager

💊 Application to use ReVanced on Android
https://revanced.app
GNU General Public License v3.0
17.81k stars 734 forks source link

bug: Add a description to auto update components #1957

Open oSumAtrIX opened 3 months ago

oSumAtrIX commented 3 months ago

Bug description

Right now, when someone opens the app for the first time, these things may be unknown, so judging to enable updates for them or not is not possible:

image

Instead, add descriptions to each component. Additionally, if the "bundle" naming concept is something we want to keep, then "ReVanced Patches" could say "Patch bundles".

An alternative would be to simplify the popup and simply ask the user if they want to keep components up to date without the ability to check the components individually. A note can be added that it'll update the following X components and that the settings can be changed later.

Acknowledgements

Axelen123 commented 3 months ago

Simply renaming "ReVanced Patches" to "Patch bundles" would be misleading because the setting is only applied to the default API bundle

oSumAtrIX commented 3 months ago

What if the API is changed.

Ushie commented 3 months ago

That would effect ReVanced Manager aswell

Axelen123 commented 3 months ago

What if the API is changed.

In that case both the current name and suggested name are unsuitable and we need a new one

oSumAtrIX commented 3 months ago

Why is Patch bundles unsuitable if the API changes

Ushie commented 3 months ago

because the setting is only applied to the default API bundle

I think it could be the default value when adding new patch bundles

oSumAtrIX commented 3 months ago

Or if it's just regarding the API bundle, use the simplifying alternative I suggested

Axelen123 commented 3 months ago

Why is Patch bundles unsuitable if the API changes

Because "Patch bundles" implies that it affects every bundle, which is not the case