Skip to content

Ensure assignee array has values before passing argument to finder

charlie ablett requested to merge 432501-cablett-issue-analytics-null-username into master

What does this MR do and why?

In GraphQL, the parameter must be present, but can be null, but in the finder, it must have a value.

Changing to remove the parameter if it does not have a value.

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

Set up your local gdk on this branch. The following query should fail on master with Internal server error but return results on this branch.

query {
  project(fullPath: "gitlab-org/gitlab") {
    issueStatusCounts(
      assigneeUsernames: null
    ) {
      all
      closed
      opened
    }
  }
}

Related to #432501

Edited by charlie ablett

Merge request reports