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

Lumen + MongoDB #166

Closed
elcobvg opened this issue Jan 7, 2018 · 3 comments
Closed

Lumen + MongoDB #166

elcobvg opened this issue Jan 7, 2018 · 3 comments
Labels

Comments

@elcobvg
Copy link

@elcobvg elcobvg commented Jan 7, 2018

Borrows heavily from the existing Laravel implementation, since I only started this because I wanted to give MongoDB a try and I thought this would be a good use case for a document-oriented db. (You know, blog post with embedded comments, tags, user profiles.) Turns out the API specs don't ask for getting article data all-at-once, but I still hope it may be useful to some.

https://github.com/elcobvg/lumen-realworld-example-app

@Cameron-C-Chapman
Copy link
Member

@Cameron-C-Chapman Cameron-C-Chapman commented Jan 20, 2018

@elcobvg so is this project complete and ready to be reviewed or still a WIP?

@elcobvg
Copy link
Author

@elcobvg elcobvg commented Jan 22, 2018

Yeah, it's ready to be reviewed. I intended to have a complete test suite, but haven't found the time yet.
Regarding #165 the repository pattern, I did consider using repositories but decided against it as it seemed a bit too much hassle: based on the API specs, they would all be incomplete implementations anyway.

EDIT: I tested the Postman collection but there seems to be a difference in interpretation as to what an ISO 8601 timestamp should look like.

@geromegrignon
Copy link
Contributor

@geromegrignon geromegrignon commented Apr 15, 2021

Hello, we moved the addition of new implementations to this website : https://codebase.show/projects/realworld
Please submit your implementation there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants