-
Notifications
You must be signed in to change notification settings - Fork 37
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
graphql inserts
works, but insert
failed with integer
field
#85
Comments
Thanks for sharing. Your issues are very well explained. I might already have an idea why this is happening. Will push a fix and notify you soon.Thanks ! |
Thanks! And why it's called tensei? |
It's called Tensei because I love anime, and this scene from Naruto Shippuden has lived in my heart ever since I watched it 😅 |
add validation to insert_many mutation, fix correct validation to integer field fix #85
add validation to insert_many mutation, fix correct validation to integer field fix #85
add validation to insert_many mutation, fix correct validation to integer field fix #85
add validation to insert_many mutation, fix correct validation to integer field fix #85
hey @archtaurus I just published a new patch that fixes this bug. Please confirm when you get a chance if you still experience this. Thanks ! |
I tested new v0.7.23. This bug is fixed and gone! 👋 |
@archtaurus v1.0 will be our very first stable release. Once we have deployments to Vercel, Netlify, tons of bug fixes, full typescript support and tons and tons of tutorials then v1 will be ready. |
Bug report
Describe the bug
graphql
inserts
works, butinsert
failed withinteger
field.Steps to reproduce the behavior
yarn create tensei-app cats
cd cats
index.js
asyarn dev
http://localhost:8810/graphql
mutation { insert_cats( objects:[{ name: "Tom", age: 3 }] ) { id } }
worksmutation { insert_cat( object:{ name: "Tom", age: 3 } ) { id } }
failed withExpected behavior
insert one cat
Screenshots
In Graphql Playground, TYPE DETAILS for
insert_cat(...):cat!
It reads
age: String
. Is this a mistake?Code snippets
N/A
System
Additional context
N/A
The text was updated successfully, but these errors were encountered: