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

prompt user to keep plugins when repairing #386

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

Aholicknight
Copy link

This pull request includes changes to enhance the user experience. The new change includes adding a user confirmation dialog to keep plugins enabled and ensuring consistent logging and progress updates.

electron_20250216_8C5YAtDVdU

User experience improvements:

  • Added a confirmation dialog in the disableAllPlugins function to ask users if they want to keep their plugins enabled. If the user chooses to keep the plugins enabled, the deletion of plugins.json is skipped.

@KingGamingYT
Copy link

Correct me if I’m wrong, but is this not just what the install option does but with an extra confirmation prompt?

@Aholicknight
Copy link
Author

Aholicknight commented Feb 14, 2025

Correct me if I’m wrong, but is this not just what the install option does but with an extra confirmation prompt?

@KingGamingYT No. When you repair betterdiscord, all plugins are disabled. For some users, this can be tiresome enabling plugins one by one, if you have tons.

@zerebos
Copy link
Member

zerebos commented Feb 14, 2025

The repair option's only feature above install is disabling plugins. For users that just need to reinstall after an update, they should use install not repair.

@Aholicknight
Copy link
Author

The repair option's only feature above install is disabling plugins. For users that just need to reinstall after an update, they should use install not repair.

@zerebos I thought if you chose install instead of repair, it will wipe out all the installed plugins and themes?

@Aholicknight Aholicknight changed the title prompt user to confirm keeping plugins enabled before deletion prompt user to keep plugins when repairing Feb 14, 2025
@KingGamingYT
Copy link

KingGamingYT commented Feb 14, 2025

I thought if you chose install instead of repair, it will wipe out all the installed plugins and themes?

Install simply re-installs BetterDiscord's files and re-injects it into the client. No plugins or themes are touched.

@Aholicknight
Copy link
Author

Install simply re-installs BetterDiscord's files and re-injects it into the client. No plugins or themes are touched.

@KingGamingYT If install re-installs BetterDiscord's files, then why is there a repair option?

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