Expose NDB id in models

[ekampf]

No description provided.

[coveralls]

Coverage decreased (-1.8%) to 96.543% when pulling 9947f38 on feature/expose_ndb_ids into f354e7c on master.

[syrusakbary]

As a suggestion, as both id's are Strings and focus on the identity of a store key maybe you can cover both use cases with just one field that receives an argument.

Something like:

store_id = graphene.String(ndb=graphene.Boolean())

def resolve_store_id(root, args, context, info):
    if args.get('ndb'):
        return store_key.id()
    return store_key.urlsafe()

😉

[ekampf]

@syrusakbary thats way nicer!

[ekampf]

@syrusakbary WDYT now?

[coveralls]

Coverage increased (+0.02%) to 98.361% when pulling 80589b4 on feature/expose_ndb_ids into f354e7c on master.

[coveralls]

Coverage increased (+0.02%) to 98.361% when pulling 80589b4 on feature/expose_ndb_ids into f354e7c on master.

[syrusakbary]

Looks good!