Skip to content

/ realworld

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

Jump to bottom

Lumen + MongoDB #166

Open
elcobvg opened this issue Jan 7, 2018 · 2 comments
Open

Lumen + MongoDB #166

elcobvg opened this issue Jan 7, 2018 · 2 comments
Labels
backend wip

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 Cameron-C-Chapman mentioned this issue Jan 9, 2018
Closed
@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 •
edited

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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backend wip
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@Cameron-C-Chapman @elcobvg
You can’t perform that action at this time.