Skip to content

Add grader field to GraphQL Submission Type - #2661

Open
Tom-Powles wants to merge 1 commit into
instructure:masterfrom
Tom-Powles:tp-add-grader-graphql
Open

Add grader field to GraphQL Submission Type#2661
Tom-Powles wants to merge 1 commit into
instructure:masterfrom
Tom-Powles:tp-add-grader-graphql

Conversation

@Tom-Powles

Copy link
Copy Markdown

Extend submission GraphQL schema with additional grader field. Brings GraphQL into parity with REST API (which includes grader_id in submission object responses). Increases usefulness of submission data in GraphQL API by reducing need to cross-reference grader data via additional queries, decreasing total number of queries and data over fetching.

Test plan

  • Create a Course with an assignment
  • Assign a teacher and a student
  • Submit as the student and mark as the teacher
  • Re-submit as the student and mark as the teacher
  • Navigate to /graphiql
  • Run a query retrieving nodes from submissionsConnection and submissionsHistoryConnection
  • Check the grader fields appear in nodes from both and has expected User fields

Extend submission GraphQL schema with additional grader field.
Brings GraphQL into parity with REST API (which includes grader_id
in submission object responses). Increases usefulness of submission
data in GraphQL API by reducing need to cross-reference grader data
via additional queries, decreasing total number of queries and data
over fetching.

Test plan

* Create a Course with an assignment
* Assign a teacher and a student
* Submit as the student and mark as the teacher
* Re-submit as the student and mark as the teacher
* Navigate to /graphiql
* Run a query retrieving nodes from submissionsConnection
and submissionsHistoryConnection
* Check the grader fields appear in nodes from both
and has expected User fields
@CLAassistant

CLAassistant commented Aug 28, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants