graphql-js
original `execute` should throw if defer/stream directives are present
#3722

Merged

original `execute` should throw if defer/stream directives are present #3722

IvanGoncharov merged 3 commits into graphql:main from yaacovCR:throw

yaacovCR changed the title ~~original execute and subscribe should throw if defer/stream are used~~ original `execute` and `subscribe` should throw if defer/stream are used 3 years ago

yaacovCR requested a review from

robrichard 3 years ago

yaacovCR added PR: bug fix 🐞

robrichard approved these changes on 2022-09-02

yaacovCR force pushed from ad7b15ea to 215c8264 3 years ago

yaacovCR force pushed from 215c8264 to 4ef5e51b 3 years ago

yaacovCR removed PR: bug fix 🐞

yaacovCR added PR: breaking change 💥

yaacovCR changed the title ~~original `execute` and `subscribe` should throw if defer/stream are used~~ original `execute` and `subscribe` should throw if defer/stream directives are present 3 years ago

robrichard approved these changes on 2022-09-28

glasser commented on 2022-09-28

original execute should throw if defer/stream are used

34ba6042

Add check to ensure using experimental execute if experimental direct…

1af828b4

yaacovCR force pushed from 4ef5e51b to d18c2222 3 years ago

yaacovCR changed the title ~~original `execute` and `subscribe` should throw if defer/stream directives are present~~ original `execute` should throw if defer/stream directives are present 3 years ago

adjust error when unexpectedly encountering experimental directives

3b00b01b

yaacovCR force pushed from 451dc393 to 3b00b01b 3 years ago

IvanGoncharov merged 522f4950 into main 3 years ago

yaacovCR deleted the throw branch 1 year ago

Reviewers

robrichard

glasser

Assignees

No one assigned

Labels

PR: breaking change 💥

Milestone

No milestone

graphql-js original `execute` should throw if defer/stream directives are present #3722 Merged

original `execute` should throw if defer/stream directives are present #3722

graphql-js
original `execute` should throw if defer/stream directives are present
#3722

Merged