Skip to content
Snippets Groups Projects

Clarify object identifiers in public GraphQL docs

Merged Luke Duncalfe requested to merge 452465-graphql-api-customer-facing-object-identifiers into master
All threads resolved!

What does this MR do?

This MR clarifies the use of object identifiers in the public GraphQL docs. The motivation of this change is #452465 (closed) and specifically #452465 (comment 1903553820) where it was identified that we had clarified how to use object identifiers in our developer docs !144142 (merged) but not the public facing docs.

Related issues

Author's checklist

If you are a GitLab team member and only adding documentation, do not add any of the following labels:

  • ~"frontend"
  • ~"backend"
  • ~"type::bug"
  • ~"database"

These labels cause the MR to be added to code verification QA issues.

Reviewer's checklist

Documentation-related MRs should be reviewed by a Technical Writer for a non-blocking review, based on Documentation Guidelines and the Style Guide.

If you aren't sure which tech writer to ask, use roulette or ask in the #docs Slack channel.

  • If the content requires it, ensure the information is reviewed by a subject matter expert.
  • Technical writer review items:
    • Ensure docs metadata is present and up-to-date.
    • Ensure the appropriate labels are added to this MR.
    • Ensure a release milestone is set.
    • If relevant to this MR, ensure content topic type principles are in use, including:
      • The headings should be something you'd do a Google search for. Instead of Default behavior, say something like Default behavior when you close an issue.
      • The headings (other than the page title) should be active. Instead of Configuring GDK, say something like Configure GDK.
      • Any task steps should be written as a numbered list.
      • If the content still needs to be edited for topic types, you can create a follow-up issue with the docs-technical-debt label.
  • Review by assigned maintainer, who can always request/require the reviews above. Maintainer's review can occur before or after a technical writer review.

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
  • Luke Duncalfe added 1 commit

    added 1 commit

    • aab6f8f8 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Luke Duncalfe requested review from @eread and removed review request for @cablett

    requested review from @eread and removed review request for @cablett

  • Luke Duncalfe changed milestone to %17.4

    changed milestone to %17.4

  • charlie ablett approved this merge request

    approved this merge request

  • Evan Read
  • Evan Read
  • Evan Read
  • Evan Read
  • Evan Read removed review request for @eread

    removed review request for @eread

  • Evan Read requested changes

    requested changes

  • Luke Duncalfe added 1 commit

    added 1 commit

    • 69441869 - Apply 3 suggestion(s) to 1 file(s)

    Compare with previous version

  • Luke Duncalfe added 1 commit

    added 1 commit

    • 442fa5de - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Luke Duncalfe requested review from @eread

    requested review from @eread

  • Luke Duncalfe
  • Luke Duncalfe added 1 commit

    added 1 commit

    • 64c4892b - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Evan Read resolved all threads

    resolved all threads

  • Evan Read approved this merge request

    approved this merge request

  • Hi @eread :wave:,

    GitLab Bot has added the Technical Writing label because a Technical Writer has approved or merged this MR.

    This message was generated automatically. You're welcome to improve it.

  • merged

  • Evan Read mentioned in commit 5238586a

    mentioned in commit 5238586a

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading