Skip to content
Snippets Groups Projects

graphqlbackend: Proper lazy loading in GitCommitResolver

Merged Warren Gifford requested to merge cloud/fix-commit-lazy-loading into main

Created by: tsenart

This fixes a bug I introduced in #15746 where not all fields of the commit would be lazy loaded.

It makes the constructor of the resolver take in a commit ID and a commit, separately, so that it's clear when we instantiate a resolver with a pre-loaded commit, vs without one (so that it gets lazy loaded).

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading