Skip to content

Geo Secondary - Update Alert

Zack Cuddy requested to merge 238993-update-geo-banner into master

What does this MR do?

Closes #238993 (closed)

This change adds a button link back to the Primary Node when on the secondary.

~"technical debt"

The Read Only alert uses the legacy flash styling, I opened #240992 (closed) to track updating it to match the Pajamas spec.

Screenshots

Read Only, Non-Geo View
before Read_Only_-_Before
after Screen_Shot_2020-08-24_at_6.51.06_PM
Read Only, Geo View (Limited Actions)
before Limited_Action_-_Before
after LImited_Actions_-_After

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team

Closes #238993 (closed)

Edited by Zack Cuddy

Merge request reports

Loading