-
-
Notifications
You must be signed in to change notification settings - Fork 500
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
Disable Package Manager Notifications #3346
Disable Package Manager Notifications #3346
Conversation
Why port it if it has been removed? |
Because while the checkbox was removed from the UI, the code for it is still present, just commented out, so I didn't know if you had plans to add it back at some point and migrated it just in case. |
I removed the code when redesigning operations, only the checkbox remains (which I forgot to remove) |
OK, I'll go remove that stuff then. I had just now noticed there was no longer any implementation details (and I wasn't transferring the settings anyway). |
Any user suspected of farming GitHub activity with crypto purposes will get banned. Submitting broken code wastes the contributors' time, who have to spend their free time reviewing, fixing, and testing code that does not even compile breaks other features, or does not introduce any useful changes. I appreciate your understanding.
This PR allows the user to disable notifications for when updates are available from certain package managers. If updates are available from other package managers, those will still be shown. The updateable packages count reflects all managers, however the list of available updates in the notification only includes those without notifications disabled.
It also updates the
CheckboxCard
setting to allow it to use the complex settings, and it ports theAlwaysElevate
and (removed)AllowParallelInstalls...
settings to use those complex settings.AlwaysElevate
settings will be transferred on open,AllowParallelInstalls...
will not be as it is disabled in the code anyway.Closes #3319