Skip to content

Convert fullPath fields to ID types

Luke Duncalfe requested to merge ld-graphql-fullPaths-to-use-IDs into master

What does this MR do and why?

This MR changes fullPath fields that use the GraphQL String type to be ID type instead, following our guidance on the use of ID types !150604 (merged).

In a follow-up MR a cop will help enforce this !152003.

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.

How to set up and validate locally

On both master and this branch, there will be no change in how the fields are returned:

For example, in GraphiQL http://gdk.test:3000/-/graphql-explorer:

{
  project(fullPath: "<FULL_PATH>") {
    issue(iid: "<IID>") {
      designCollection {
        designs {
          nodes {
            fullPath
          }
        }
      }
    }   
  }
}
Edited by Luke Duncalfe

Merge request reports