Skip to content

Add graphql-go with null inputs support #9

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

Merged
merged 8 commits into from
Mar 25, 2020
Merged

Add graphql-go with null inputs support #9

merged 8 commits into from
Mar 25, 2020

Conversation

mpenick
Copy link
Contributor

@mpenick mpenick commented Mar 24, 2020

No description provided.

@mpenick
Copy link
Contributor Author

mpenick commented Mar 24, 2020

To use:

GOPRIVATE="github.com/riptano/*" go get

Example:

Create the table

mutation {
  createTable(name: "books", 
    partitionKeys: [{name: "title", type: {basic: TEXT}}]
  	values:[{name: "pages", type: {basic:INT}}])
}

Insert value with a null

mutation {
  insertBooks(data: {
    title:"abc",
    pages: null
  }) {
    applied
  }
}

Select value

query{
  books(data: {
    title:"abc"
  }) {
    values {
      title
      pages
    }
  }
}

@mpenick
Copy link
Contributor Author

mpenick commented Mar 24, 2020

Having a private dependency is not good for CI. I'll look into moving this to datastax instead of riptano. Or we can use just use my public branch...

It was a pain, but I got CI working on travis CI. Still might be easier to just put it under datastax.

go.mod Outdated
@@ -13,3 +13,5 @@ require (
gopkg.in/inf.v0 v0.9.1
gopkg.in/yaml.v2 v2.2.8 // indirect
)

replace github.com/graphql-go/graphql => github.com/riptano/graphql v0.7.9-null
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good idea

@jorgebay
Copy link
Contributor

NICE!

we can use just use my public branch

I don't see a problem in using a public repo, under riptano, datastax or under your name, it will make our lives easier and it should be OSS eventually.

@mpenick mpenick merged commit bf5b585 into master Mar 25, 2020
@mpenick mpenick linked an issue Mar 26, 2020 that may be closed by this pull request
@mpenick mpenick deleted the null_support branch April 24, 2020 13:29
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.

Support null literals as a value for arguments and input object fields
2 participants