Skip to content
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 meta fields to account schema #9

Open
advishnuprasad opened this issue Mar 7, 2019 · 2 comments
Open

Add meta fields to account schema #9

advishnuprasad opened this issue Mar 7, 2019 · 2 comments

Comments

@advishnuprasad
Copy link

advishnuprasad commented Mar 7, 2019

Meta fields created_by, updated_by, created_at and updated are missing in account entity.
Were they missed on purpose?

@abhishek-ram
Copy link
Contributor

Yes @advishnuprasad the idea is to add the bare minimum properties and allow users to extend if needed, right @subhashb ?

@subhashb
Copy link
Contributor

subhashb commented Mar 7, 2019

That would be true for the Protean core, yes.

But I see the following aspects in Authentic that are different:

  • Authentic is an implementation of Protean, and will probably not be inherited. Users will use it for the out-of-the-box functionality, so will probably not tamper with the models.
  • Authentic is all about security, and the four meta fields are required to ensure backtracking and audits.
  • If Authentic were made available as a "Service" on the cloud, these meta fields would be necessary.

Let us include the four meta fields as part of Authentic. As usual, the correct way would be to create a custom Base Entity, inheriting from Entity, and add the audit fields to it, before inheriting all other entities from the customized Base Entity.

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

No branches or pull requests

3 participants