Skip to content

Prep work for refactoring the pipelines table to use GlTable [RUN ALL RSPEC] [RUN AS-IF-FOSS]

Payton Burdette requested to merge pb-pipelines-table-refactor-prep-work into master

What does this MR do?

Preparation to refactor the pipelines table to use GlTable. I have a larger MR open for a bigger overview here !54619 (closed)

We decided to use a FF here and this MR edits some of the components before hand. To keep the MR's smaller.

FYI this flag is not meant to be turned on right now as the UI will be broken. The following MRs will do the entire refactor leading up to the flag being able to be turned on.

Related to: #321257 (closed)

Screenshots (strongly suggested)

With flag disabled

Screen_Shot_2021-02-24_at_9.15.51_AM

Screen_Shot_2021-02-25_at_8.48.59_AM

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
Edited by Payton Burdette

Merge request reports