Introducing lastUpdated in systemSetting

Mansi_Singhal · 21 January 2015 09:55

Hey Lars,

We are beginning to work on a patch to introduce lastUpdated field in systemSetting.

We would like this patch to be applied on trunk.

Do you see any issues with the same?

···

Thanks,

Mansi Singhal

Lars · 21 January 2015 15:57

Hi Mansi,

that sounds fine, go ahead. Can you also add “created” in the process? And could you do this to UserSetting as well to keep it consistent?

Ideally I think we should make SystemSetting an IdentifiableObject, and use the standard stores, controllers, etc. That is a bit more work. Introducing created and lastUpdated is a step in the right direction, so its great if you can do that.

best regards,

Lars

···

On Wed, Jan 21, 2015 at 10:55 AM, Mansi Singhal msinghal@thoughtworks.com wrote: