build-canaries / nevergreen

:baby_chick: A build monitor with attitude
https://nevergreen.io
Eclipse Public License 1.0
209 stars 38 forks source link

Treat "Allow audio notifications" as a personal setting #340

Closed GentlemanHal closed 1 year ago

GentlemanHal commented 1 year ago

Importing configuration shouldn't override the "Allow audio notifications" setting.

The "Allow system notifications" is already filtered out of imports and exports. The simplest thing to do is copy that code to exclude audio notifications as well, however this is a good opportunity to improve the code and make "personal settings" more obvious.