proteanhq / authentic

Comprehensive Authentication Package
BSD 3-Clause "New" or "Revised" License
1 stars 3 forks source link

Add meta fields to account schema #9

Open advishnuprasad opened 5 years ago

advishnuprasad commented 5 years ago

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

abhishek-ram commented 5 years ago

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

subhashb commented 5 years ago

That would be true for the Protean core, yes.

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

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.