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

Siena-MongoDB implementation #8

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

Conversation

thagikura
Copy link

I implemented Siena-MongoDB.

I didn't know there is already mongodb branch in Siena.
If there is no problem, please try it.

@mandubian
Copy link
Owner

Hi Takeshi!

This is really great! Thanks a lot!
I'm going to investigate your code!
My mongo branch provided already a few features but not all.
I had also rewritten lots of unit tests for Mongo...

Did you implement everything or just a part of it?

Best regards
Pascal

@thagikura
Copy link
Author

Hi Pascal!
Thank you for your reply.

Actually I implemented part of it. Those source codes are based on that post on Google Code about a year ago.
( http://code.google.com/p/siena/issues/detail?id=6 )

But that codes didn't work under the current version of Siena APIs.
So I fixed and added JUnit test codes to it.

Please investigate it, thanks.

Best regards
Takeshi

@mandubian
Copy link
Owner

I'm going to fetch your fork directly and work from it because I don't want to merge your pull request into the trunk but I'd prefer working on the mongodb branch!

@thagikura
Copy link
Author

Ok. thanks!

@thagikura
Copy link
Author

Hi Pascal,

How is the investigation going?
If there is a slightest thing to ask me, please let me know, thanks!

@mandubian
Copy link
Owner

I'm late :D
Sorry but I had no time to work on it!

Pascal

On Sun, Nov 27, 2011 at 7:56 AM, Takeshi Hagikura <
[email protected]

wrote:

Hi Pascal,

How is the investigation going?
If there is a slightest thing to ask me, please let me know, thanks!


Reply to this email directly or view it on GitHub:
#8 (comment)

@mrcritical
Copy link

Still waiting for Mongo support. I really don't like Morphia, which is the only other real option for Play!.

@adpande
Copy link

adpande commented Oct 22, 2012

Any updates on mongodb support?

Regards,

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.

4 participants