Comments (10)
Yes, you are right! There is a fix hidden in this commit, but it hasn't been released yet. I'll do a release soon, it will most likely fix your problem.
from graffiti-mongoose.
Sadly I'm already using that version and doesn't seem to fix the issue
from graffiti-mongoose.
hmm... strange :/ Let me take a look at it in a few hours (just got home from the SW premier, it's already 4am :)) It would be nice if you could test it too, so you might find where the error comes from.
from graffiti-mongoose.
@dzannotti Okay, I think it's fixed now. Try out version 5.0.1
!
from graffiti-mongoose.
@tothandras oh wow, i wasn't expecting such a quick fix! Was going to write some tests for it and PR tonight with a fix, but you got was waaay ahead of me. Will test it out, that's great, i do wonder if a safer way to figure out if the id is of mongo or global relay is to verify if it ends with =
as originally was checking along other things, graphql-relay uses base64 encoding to it'll always end with =
while to the best of my knowledge no mongoid will do that
from graffiti-mongoose.
Yes, you are right! I was thinking about that before, but as far as I know
=
at the end indicates duplicated characters or something. I wasn't brave
enough to use this, but at the same time I couldn't find an ID without it.
:)
On Fri, Dec 18, 2015, 14:58 Daniele Zannotti [email protected]
wrote:
@tothandras https://github.com/tothandras oh wow, i wasn't expecting
such a quick fix! Was going to write some tests for it and PR tonight with
a fix, but you got was waaay ahead of me. Will test it out, that's great, i
do wonder if a safer way to figure out if the id is of mongo or global
relay is to verify if it ends with = as originally was checking along
other things, graphql-relay uses base64 encoding to it'll always end with
= while to the best of my knowledge no mongoid will do that—
Reply to this email directly or view it on GitHub
#75 (comment)
.
from graffiti-mongoose.
To the best of my knowledge =
is used as padding to indicate that the last character encoded is 8 or 16 bits, so in theory not every base64 string would end with a =
however i never came across one that didn't have it. Not had the chance to try the fix yet however after reporting this issue, it seem to have appear randomly on other string fields (not necesarly arrays). WIll upgrade and investigate as soon as i can
from graffiti-mongoose.
@dzannotti You are most certainly right, I don't trust my memories. :) Update me if you tried it!
from graffiti-mongoose.
yep confirmed working in 5.0.1, thank you for this!
from graffiti-mongoose.
@dzannotti Thanks for the bug report!
from graffiti-mongoose.
Related Issues (20)
- Implicit dependency on babel-polyfill HOT 9
- Cannot set currently-valued fields to null in update mutation HOT 2
- How to make a custom field: id? HOT 10
- "TypeError: Cannot read property 'forEach' of undefined" when embedded object field is a reference HOT 6
- Custom query thunk - accepts only InputObject types HOT 3
- Question on customQueries HOT 7
- Example project doesnt work with [email protected] and [email protected] HOT 1
- Async hook support HOT 2
- How to use Express Middleware for Authentication/Authorization. HOT 4
- Get specific field validation error?
- How to mutate nested Schema? HOT 3
- What is different between id and _id HOT 1
- UNMET PEER DEPENDENCY HOT 1
- Maintainer Volunteer Thread HOT 1
- Mongoose ObjectId and GraphQL id is not the same. How to link them? HOT 5
- It is possible to add some description to schema and to be reflected in the docs?
- Does graffiti-mongoose works with react-apollo ? HOT 2
- Custom mutations?
- Custom queries/mutations HOT 1
- setTypeFields bug
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from graffiti-mongoose.