Skip to content

Geo 2.0 Regression - Multiple Secondaries

Zack Cuddy requested to merge 330001-geo-beta-multiple-secondaries into master

What does this MR do?

Fixes #330001 (closed)

This change decouples the Geo Sites title sections with the nodes. This allows for the title sections to not be duplicated in the loop. These title sections are to act as "containers" so having multiple defeated the purpose.

This change is also behind a feature flag (:geo_nodes:beta)

How to test

  1. Set up a Geo Primary/Secondary GDK
  2. Fetch the current branch
  3. Activate the Feature Flag on your Primary GDK (:geo_nodes_beta)
  4. Navigate to http://127.0.0.1:3000/admin/geo/nodes_beta
  5. View UI

Screenshots (strongly suggested)

Before After
bug Before_-_Bug After_-_Fix

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

Related to #330001 (closed)

Edited by Zack Cuddy

Merge request reports

Loading