Skip to content

Correctly handle nil assignee arguments

charlie ablett requested to merge 432501-cablett-null-assignee-counts into master

What does this MR do and why?

This MR adjusts some logic in our parameter normalisation in the GraphQL resolvers in !152127 (merged) and !151639 (merged)

Changelog: fixed

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

Both these queries should not error locally:

GraphQL queries:

query {
  group(fullPath: "gitlab-org") {
    flowMetrics {
      issuesCompletedCount(
        assigneeUsernames: null
        from: "2022-01-01"
        to: "2022-02-02"
      ) {
        value
      }
    }
  }
}
query {
  project(fullPath: "gitlab-org/gitlab") {
    issueStatusCounts(
      assigneeUsernames: null
    ) {
      all
      closed
      opened
    }
  }
}

Related to #432501 (closed)

Merge request reports