Skip to content
This repository has been archived by the owner on Dec 20, 2024. It is now read-only.

use body.extensions #292

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

use body.extensions #292

wants to merge 1 commit into from

Conversation

Rowadz
Copy link

@Rowadz Rowadz commented Apr 5, 2023

when sending an extensions object with a get request the package expects an extensions in the query params, but when doing a post request it expects an object called extension in the payload

I believe this is a bug.

if (isHttpMethod("GET", method)) {
operationName = queryParams.operationName;
query = queryParams.query;
variables = queryParams.variables;
extensions = queryParams.extensions;
} else if (isHttpMethod("POST", method)) {
operationName = body?.operationName;
query = body?.query;
variables = body?.variables;
extensions = body?.extension

I opened this PR to fix it ツ

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants