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

Add internal fields in model #300

Closed
jimlambie opened this Issue Jun 8, 2017 · 2 comments

Comments

Projects
3 participants
@jimlambie
Copy link
Member

jimlambie commented Jun 8, 2017

Internal fields such as createdAt are added to the incoming data in the controller, so if an insert is performed using the model instance, no internal fields are added.

Suggest we add these in the model, not in the controller.

/cc @eduardoboucas thoughts?

@eduardoboucas

This comment has been minimized.

Copy link
Member

eduardoboucas commented Jun 8, 2017

Absolutely. Perhaps this is something we can tackle along with #141 on the next major release?

@jimlambie jimlambie added this to Backlog in API Version 3.0 Jun 12, 2017

@fra967

This comment has been minimized.

Copy link
Member

fra967 commented Jul 7, 2017

why don't we use just one field (whose name is configurable) and then all internal fields are elements of that single field?
alternatively, please make the prefix configurable

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