Problem/Motivation

All plugins in the module currently pass ?string around for SDL definitions. In case of errors it can be difficult to know where the error originated.

Steps to reproduce

Proposed resolution

Ensure that \GraphQL\Language\Source is consistently used and contains location information.

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork graphql-3586836

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

kingdutch created an issue. See original summary.

kingdutch opened merge request !84

kingdutch’s picture

kingdutch
he/him
Primary language English
commented
Status: Active » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • kingdutch committed e5836104 on 5.x 
    feat: #3586836 Require use of `Source` for loaded SDL
kingdutch’s picture

kingdutch
he/him
Primary language English
commented
Assigned: kingdutch » Unassigned

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.