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

Deprecate Ecto.Model.Callbacks #1104

Closed
josevalim opened this Issue Nov 28, 2015 · 2 comments

Comments

Projects
None yet
1 participant
@josevalim
Member

josevalim commented Nov 28, 2015

We will migrate its API to the Ecto model. We need to remove model references to something else. We also need to provide an alternative for callbacks.

@josevalim josevalim added this to the v1.1 milestone Nov 28, 2015

@josevalim josevalim self-assigned this Nov 28, 2015

@josevalim

This comment has been minimized.

Show comment
Hide comment
@josevalim

josevalim Nov 30, 2015

Member

Questions:

  • Provide an alternative mechanism for repository events (prepare_query, prepare_insert, prepare_update, ...)
  • Provide a schema to configure the struct after load and before dump
  • Provide a mechanism for explicit after hooks
Member

josevalim commented Nov 30, 2015

Questions:

  • Provide an alternative mechanism for repository events (prepare_query, prepare_insert, prepare_update, ...)
  • Provide a schema to configure the struct after load and before dump
  • Provide a mechanism for explicit after hooks
@josevalim

This comment has been minimized.

Show comment
Hide comment
@josevalim

josevalim Nov 30, 2015

Member

We will also need to move the functions from Ecto.Model to Ecto (emit warnings probably only on 2.0). So ideally Ecto.Model by 2.0 is a module with deprecated functions that uses only Ecto.Schema and imports itself.

Member

josevalim commented Nov 30, 2015

We will also need to move the functions from Ecto.Model to Ecto (emit warnings probably only on 2.0). So ideally Ecto.Model by 2.0 is a module with deprecated functions that uses only Ecto.Schema and imports itself.

@josevalim josevalim changed the title from Deprecate Ecto.Model to Deprecate Ecto.Model.Callbacks Dec 2, 2015

@josevalim josevalim closed this Dec 2, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment