Todo: Support Ecto 3

[mmartinson]

Ecto 3 is coming in the next couple days and with it the recommended upgrade path for those using it for database interaction is to change dependency from ecto to ecto_sql.

I think to support backwards compatibly for presumably a long time we'll want to introduce an optional dependency.

Reference commit and announcement

[idyll]

One of the changes that's needed is in cast_attachments

The allowed params in:

cast(changeset_or_data, arc_params, allowed)

are converted into strings. Which triggers all sorts of warnings because they should be atoms.

[nathany]

It seems that the test suite doesn't require ecto_sql.

#101 amends #99 for Ecto 3.0 final. @tarzan resolved the warnings.

Do you want to support Ecto 2.x and 3.x simultaneously, or just release a new major version that requires Ecto 3? [people can always use an old version of arc_ecto]

[williamweckl]

Any updates on this?

[dbernheisel]

I think this issue is resolved with #103

[williamweckl]

@dbernheisel Yes it is!