Skip to content
Snippets Groups Projects

Convert old GraphQL types to newer type names (Part 8) [RUN AS-IF-FOSS]

All threads resolved!

What does this MR do?

During conversion to using the new graphql interpreter, I ran across this same issue: https://github.com/rmosolgo/graphql-ruby/issues/2784

So doing all the type conversions here in preparation for !27536 (merged)

  • Convert GraphQL::ID_TYPE to GraphQL::Types::ID
  • Convert GraphQL::INT_TYPE to GraphQL::Types::Int
  • Convert GraphQL::STRING_TYPE to GraphQL::Types::String
  • Convert GraphQL::BOOLEAN_TYPE to GraphQL::Types::Boolean

Screenshots or Screencasts (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #336292 (closed) and #210556 (closed)

Edited by Nikola Milojevic

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