Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bad JSON stringify in get requests #60

Merged
merged 1 commit into from
Jun 17, 2017
Merged

Conversation

jeroenbe
Copy link
Contributor

Bad GET requests were sent

Not sure if this fixes #52 but it's worth checking. It definitely fixes my issue. I'm not sure why the query parameter didn't cause any issues as the object given there wasn't (and still isn't) JSON.stringified.

@skevy skevy merged commit c28bce7 into skevy:master Jun 17, 2017
madzhup added a commit to madzhup/graphiql-app that referenced this pull request Oct 16, 2017
* source:
  Ok npm wants me to commit this
  Revert "add package lock file (skevy#86)"
  take shell straight of electron and updated button/links (skevy#87)
  add package lock file (skevy#86)
  Update readme (skevy#85)
  add the close query menu item for non-macOS (skevy#71)
  Add Origin header for CORS support (skevy#66)
  Update productName
  Add release command
  Update .gitignore
  Package with Electron Builder, add windows + linux packages
  Fix bad JSON stringify in get requests (skevy#60)
  Update to Babel 6 + GraphiQL 0.10.2/GraphQL 0.10.1

# Conflicts:
#	package.json
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.

Variables are invalid JSON with GET request - regression
2 participants