Skip to content

Conversation

@ravanscafi
Copy link

@ravanscafi ravanscafi commented Nov 14, 2018

Easy, powerful and ultrafast ODM for PHP7 build on top of the new mongodb driver.

Mongolid

@ravanscafi
Copy link
Author

@jmikola thanks for the review, I've updated it and gave more context.

Copy link
Member

@jmikola jmikola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies for losing track of this. I've made some final suggestions.

@terakilobyte, @zach-carr: feel free to merge this with the suggested changes.

Built on top of `Mongolid`, it provides many integrations with the framework,
while leveraging some core features of it. You might find some similarities
with `Eloquent`, but this library is not a replacement for it, as it tries
to follow MongoDB concepts.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested rephrase:

Built on top of the Mongolid ODM, this library provides integrations with the framework and some of its core features. There are some similarities with Eloquent but this library is designed specifically with MongoDB concepts in mind.

- `Mongolid <https://github.com/leroy-merlin-br/mongolid>`_: Easy, powerful,
and ultrafast ODM for PHP and MongoDB. Implements ActiveRecord design
pattern accounting for MongoDB concepts. It supports embedded and referenced
documents.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested rephrase:

Easy, powerful, and ultrafast ODM for PHP and MongoDB. Implements both ActiveRecord and DataMapper design patterns and supports embedded and referenced documents.

jmikola added a commit to jmikola/docs-ecosystem that referenced this pull request Aug 11, 2020
This is based on mongodb#463 with suggested changes applied.
@jmikola jmikola mentioned this pull request Aug 11, 2020
@jmikola
Copy link
Member

jmikola commented Aug 11, 2020

Sorry that we lost track of this. I've incorporated the suggested changes and opened #669, so hopefully that can be merged soon.

MongoCaleb pushed a commit that referenced this pull request Aug 11, 2020
* Add Mongolid library

This is based on #463 with suggested changes applied.

* Revise Mongolid description
@MongoCaleb MongoCaleb closed this Aug 11, 2020
@MongoCaleb
Copy link
Contributor

Closed as per @jmikola

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