Use global IDs when exposing GraphQL resources
What does this MR do?
This exposes all fields named id
as GlobalIDs so they can be used
across our entire GraphQL implementation.
When the objects loaded are ApplicationRecord
s. We'll use our
existing batchloading to find them. Otherwise, we'll fall back to the
default implementation of GlobalID
: Calling the .find
method on
the class.
It also fixes the non-unique usages of GraphQL::ID_TYPE
.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/62650
Does this MR meet the acceptance criteria?
Conformity
-
Changelog entry -
Documentation created/updated or follow-up review issue created -
Code review guidelines -
Merge request performance guidelines -
Style guides - [-] Database guides
-
Separation of EE specific content
/copy_metadata https://gitlab.com/gitlab-org/gitlab-ce/issues/62650
Edited by Bob Van Landuyt