Skip to content

GraphQL: Update merge train fields

Vlad Wolanyk requested to merge 473241-move-merge-train-fields-to-mergetraincar into master

What does this MR do and why?

Updates MergeTrain related Graphql fields

  • moves merge_train_index from MergeRequestType to MergeTrains::CarType and renames the field to :index
  • deprecates merge_train_count from MergeRequestType in favour of using count available through the CarType.connection_type available through cars on MergeTrains::TrainType

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Merge Train Car Count

Before After
Screenshot_2024-08-14_at_12.26.51_PM Screenshot_2024-08-14_at_12.30.57_PM

Merge Train Index

Before After
Screenshot_2024-08-14_at_12.32.40_PM Screenshot_2024-08-14_at_3.12.31_PM

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Related to #473241 (closed)

Edited by Vlad Wolanyk

Merge request reports