revanced-manager icon indicating copy to clipboard operation
revanced-manager copied to clipboard

bug: Add a description to auto update components

Open oSumAtrIX opened this issue 1 year ago • 8 comments

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

  • [X] This issue is not a duplicate of an existing bug report.
  • [X] I have chosen an appropriate title.
  • [X] All requested information has been provided properly.
  • [X] The bug is only related to ReVanced Manager

oSumAtrIX avatar Jun 22 '24 21:06 oSumAtrIX

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

Axelen123 avatar Jun 29 '24 23:06 Axelen123

What if the API is changed.

oSumAtrIX avatar Jun 29 '24 23:06 oSumAtrIX

That would effect ReVanced Manager aswell

Ushie avatar Jun 29 '24 23:06 Ushie

What if the API is changed.

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

Axelen123 avatar Jun 29 '24 23:06 Axelen123

Why is Patch bundles unsuitable if the API changes

oSumAtrIX avatar Jun 29 '24 23:06 oSumAtrIX

because the setting is only applied to the default API bundle

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

Ushie avatar Jun 29 '24 23:06 Ushie

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

oSumAtrIX avatar Jun 30 '24 00:06 oSumAtrIX

Why is Patch bundles unsuitable if the API changes

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

Axelen123 avatar Jun 30 '24 00:06 Axelen123