Skip to content
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

Add google cloud datastore implementation #136

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

amura2406
Copy link

Hi, I just added google cloud datastore implementation for my own usage, just thought that you might want to merge it in your main repository.

Feedbacks are welcome.

ADD google cloud datastore index definition [mandatory for test]
@adrai
Copy link
Contributor

adrai commented Oct 29, 2018

Can index.yaml be moved somewhere else? i.e. test directory?

@amura2406
Copy link
Author

yes, it can
should I move it ?

@adrai
Copy link
Contributor

adrai commented Oct 29, 2018

yes

@amura2406
Copy link
Author

already move the file to test folder

@adrai
Copy link
Contributor

adrai commented Oct 30, 2018

Can you also extend the readme?

@adrai
Copy link
Contributor

adrai commented Oct 30, 2018

btw: Can you please use the use function Store.use instead of require (like the other db implementations)?

update README to cover "cloud datastore"
@amura2406
Copy link
Author

done

@adrai
Copy link
Contributor

adrai commented Oct 31, 2018

Is the datastore “testable” via travis or should we skip it in the tests?

@amura2406
Copy link
Author

I think so, but I haven't got lucky with cloud datastore emulator in the past few days

@adrai
Copy link
Contributor

adrai commented Nov 12, 2018

ok, then perhaps it would be better to skip it here and here

@kojuka
Copy link

kojuka commented Dec 17, 2020

@amura2406 any updates to this? I'd also like to use cloud datastore.

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.

3 participants