Skip to content

Pipeline Graph Structural Update: Adding Accessors

What does this MR do?

This MR is the fourth of the MRs related to #276949 (closed).

It adds accessors for components that are not behind the flag but are consumed by both the GraphQL and REST components. This accounts fields that differ from one another in naming or position.

  • app/assets/javascripts/pipelines/components/graph/job_item.vue
  • app/assets/javascripts/jobs/mixins/delayed_job_mixin.js
  • app/assets/javascripts/pipelines/components/graph/linked_pipeline.vue

The goal with the accessor is to have an obvious focus for removing unneeded accessors when the graph is completely replaced. The delayed_job_mixin does not use these accessors because, as it is used by multiple components, it will need to permanently access properties on both types.

This MR is successful to the extent does not change any layout or behavior.

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Related to #276949 (closed)

Merge request reports

Loading