-
Notifications
You must be signed in to change notification settings - Fork 172
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
Expose and prefix JSON type to avoid conflict with other libraries #403
Expose and prefix JSON type to avoid conflict with other libraries #403
Conversation
ad15d4e
to
1b66552
Compare
Current coverage is 92.66% (diff: 100%)@@ master #403 diff @@
==========================================
Files 11 11
Lines 368 368
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
Hits 341 341
Misses 27 27
Partials 0 0
|
@@ -5,5 +5,6 @@ module.exports = { | |||
typeMapper: require('./typeMapper'), | |||
attributeFields: require('./attributeFields'), | |||
simplifyAST: require('./simplifyAST'), | |||
relay: require('./relay') | |||
relay: require('./relay'), | |||
GraphQLJSON: require('./types/jsonType') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just JSONType maybe? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure! I've updated the pr :)
1b66552
to
68db423
Compare
Is there something preventing this from being merged? |
@AlexGaspar Nope, not at all, i forgot to merge & release, will do it tomorrow. |
v5.0.0 |
Implement #343 ideas