PacktPublishing / ASP.NET-Core-2-and-Angular-5

ASP.NET Core 2 and Angular 5, published by Packt
MIT License
79 stars 75 forks source link

Clarification: meaning of LastModifiedDate #26

Closed samiroquai closed 6 years ago

samiroquai commented 6 years ago

Hi,

Just to make sure:

If these are true, then when you update a Quiz (Quiz Controler, Post Method, related to issue #17 ) do you set the LastModifiedDate to the CreatedDate? (see code below)

// properties set from server-side

quiz.LastModifiedDate = quiz.CreatedDate;

Shouldn't it be more like the following?

// properties set from server-side

quiz.LastModifiedDate = DateTime.Now;
Darkseal commented 6 years ago

Yeah, definitely: that was an evident oversight. Luckily enough, that's not a critical errata.

Thank you for pointing it out!