Skip to content

Expose GraphQL Global ID on AwardEmoji connection

Kushal Pandya requested to merge kp-expose-emoji-reaction-id-graphql into master

What does this MR do and why?

Exposes GraphQL Global ID on AwardEmoji connection to make it consistent with other connection types where global ID is always available.

Note: We're disabling @graphql-eslint/require-id-when-available rule in all the GraphQL files where awardEmoji is used, since with addition of id field in this MR, linter would complain of missing id in all usages but we cannot start using id within same MR because of multi-version compatibility. These rules will stay disabled within %16.2 and will be enabled again in %16.3 when we can start using id in all the usages of awardEmoji.

Screenshots or screen recordings

Not applicable.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Kushal Pandya

Merge request reports