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

bug: Dangerous option bring offered without warning

Open validcube opened this issue 11 months ago • 1 comments

Bug description

You can regen your keystore, reset patch select, options without any warning.

This PR extends on #2534

The idea is to:

  1. Let user know what they're doing
  2. Let user know the consequence
  3. If it a really really bad idea, discourage the user

Suggested design is to use something like this

https://www.figma.com/design/ONBQjUCpPTvcMeo5oZUMVE/ReVanced-Concept?node-id=57265-104402&t=XWFWhYQN3CPylIKi-1

Image

Version of ReVanced Manager and version & name of app you are patching

All (Compose)

Installation method

None

ReVanced Manager logs

abcdefghijklmnopqrstuvwsyz

Patch logs

abcdefghijklmnopqrstuvwsyz

Acknowledgements

  • [x] I have checked all open and closed bug reports and this is not a duplicate.
  • [x] I have chosen an appropriate title.
  • [x] All requested information has been provided properly.
  • [x] The bug is only related to ReVanced Manager.

validcube avatar May 08 '25 19:05 validcube

From a user which has no understanding on the dev process, I think reset patch selections/options description should be clarified. Personally, I don't understand what pressing that button will do and how it will change the manager.

What are these buttons doing?

brosssh avatar May 09 '25 06:05 brosssh

Closing since a linked PR was merged

Axelen123 avatar Jun 10 '25 14:06 Axelen123