Skip to content

[WIP] Ecto 2 compatibility #84

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

Closed
wants to merge 53 commits into from
Closed

[WIP] Ecto 2 compatibility #84

wants to merge 53 commits into from

Conversation

michalmuskala
Copy link
Collaborator

@michalmuskala michalmuskala commented Jun 24, 2016

This updates driver to the new db_connection based version and ecto to 2.0.

@agusguerra10
Copy link

Hey Michal, any news about this upgrade? I'm trying to use scrivener_ecto package but it requires Ecto 2.0. Thanks!

@ankhers ankhers mentioned this pull request Aug 10, 2016
11 tasks
@Morganjackson
Copy link

@michalmuskala is this still in the works? I don't know how much work is involved to finish this but I would be happy to kick in $200 if it helps.

Looks like there are a few people wanting this so maybe we could setup something like patreon?

@Fudoshiki
Copy link

+1

@ankhers
Copy link
Collaborator

ankhers commented Aug 17, 2016

For anyone that is waiting for this. Please look at #91. I am actively working on it.

* Mongo does not support returning
* Mongo does not support joins
* Mongo does not support composite keys
Mongo wants you to be explicit about not wanting the ID field returned
@michalmuskala
Copy link
Collaborator Author

As mentioned above, the work continues in a separate PR #91

Unfortunately, updating this project is not a priority for me at the moment because of other obligations. If anybody would like to see this happen, I strongly encourage you to give @ankhers a hand and help with his PR.

@ankhers ankhers mentioned this pull request Feb 16, 2017
@rsilvestre
Copy link

Hello, stupid question, but when do you think it's going to be fixed?

Regards,

@ankhers
Copy link
Collaborator

ankhers commented Feb 23, 2017

@rsilvestre This is already in a good place. Myself and others are already using it in production. All you need to do is point your dependency to this repo and branch. Something along the lines of

{:mongodb_ecto, github: "michalmuskala/mongodb_ecto", branch: "ecto-2"}

That should get you started.

I believe the reason there has not been an actual release yet is because we are still touching up documentation, and there are a couple fixes that need to be applied to ecto. Unfortunately, I am unsure on the timeframe of an actual release. If you, or anyone else for that matter, would like to be notified of a release, make sure you are subscribed to this PR and you will get a notification when it is merged.

@johnnyshields
Copy link

Hi @ankhers is there a list of fixes which we're waiting on to be applied to Ecto?

@ankhers
Copy link
Collaborator

ankhers commented Mar 6, 2017

@johnnyshields Unfortunately I do not have that list. @michalmuskala would have a better idea of the changes that are required.

tiagopog and others added 2 commits March 7, 2017 18:51
* Fix pattern matching error when creating uniqueness index

* Ensure returning the expected tuple for uniqueness validations

* Remove unnecessary string interpolation
* updated travis.yml

* requires Elixir ~> 1.2

* restored recipients
@ankhers
Copy link
Collaborator

ankhers commented Mar 8, 2018

I am going to close this in favour of the 2.1 branch release.

@ankhers ankhers closed this Mar 8, 2018
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.