Skip to content

Design Repo Sync Status - Basic View

Zack Cuddy requested to merge design-repo-base into master

Why WIP

  • API MR must be merged (target branch) - !19117 (merged)
  • Hide behind feature flag: Feature.enable(:enable_geo_design_sync)
  [x] `GlCard` component to replace `BCard` - gitlab-ui!871 (merged)

      - Replaced with Bootstrap CSS classes, will need to be replaced when this MR is merged.

Important Notes

This MR was originally created in !19116 (closed). This is an effort to split that work into 3 separate MRs. This one being the first.

MRs to add additional functionality as follows in order:

  1. !19589 (merged)
  2. !19590 (merged)

This MR is work towards #34336 (closed)

What does this MR do?

This MR adds the MVC to show a list of all Design Repositories' Sync Status in a Secondary Node of Gitlab Geo.

Describe in detail what your merge request does and why.

This MR contains the scaffolding for the View into Design Repositories' Sync Status. It is a basic, server side paginated list of all design repos. In the state this MR leaves them, there is nothing actionable or any efficient way of filtering the list (those features are in the following MRs).

I apologize in advance for the size of this initial MR. There is a lot of files required to get a barebones Vuex Store up and running. This MR was made in an attempt to make the initial MR smaller, if there is a better way to break this down even further please let me know.

Screenshots

Feature Flag Disabled (Nav): Design_Sync_Disabled__Nav_

Feature Flag Disabled (View): Design_Sync_Disabled__View_

Feature Flag Enabled (Nav): Design_Sync_Enabled__Nav_

Feature Flag Enabled (View) - GIF of basic pagination:

Geo_Designs_Base

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

cc: @ekigbo

Edited by Zack Cuddy

Merge request reports