-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
logic for shield settings is located in multiple places and none of them are shareable with android #5416
Comments
QA - this affects all shield settings for both defaults and site settings. Extensive unit tests have been added, but it still needs a pass for all settings. |
Verification passed on
Verified Shields Panel
NTP stats
Global Shields Settings
Verified following Upgrade scenarios from 0.68.141
Verification passed on
Verified Shields Panel
NTP stats
Global Shields Settings
Verified following Upgrade scenarios from 0.68.141
Verified passed with
Shields Panel
NTP stats
Global Shields Settings
Verified following Upgrade scenarios from 0.68.141
|
Shields setting logic needs to be consolidated so it is consistent everywhere
The text was updated successfully, but these errors were encountered: