search

razzius / weave

Source code for a platform to connect mentors with students
MIT License
7 stars 2 forks source link

Add "date last updated" to the faculty profile #143

Closed jjchen20 closed 4 years ago

jjchen20 commented 4 years ago

Will make it easier to identify how recently a faculty has created or updated their profile.

razzius commented 4 years ago

Looks like this image

jjchen20 commented 4 years ago

Looks good! Thanks! Is there a way for the admin to select whether any changes on the admin end should be reflected in the profile last updated date? I am imagining we may want to edit capitalization, but not want the date to change. Or, if we help a faculty make a change, we may want that content change to be reflected in the last updated date.

On Thu, Dec 19, 2019 at 1:05 PM Razzi Abuissa notifications@github.com wrote:

Looks like this [image: image] https://user-images.githubusercontent.com/2244895/71209389-0a704480-2271-11ea-9843-7d7589ecdc52.png

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/razzius/hms-weave/issues/143?email_source=notifications&email_token=AKOFUUKBFD37DFR3VJIJRCDQZPO3JA5CNFSM4J3T2ETKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHLA6OA#issuecomment-567676728, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKOFUULTO5T4RJAPJESD45TQZPO3JANCNFSM4J3T2ETA .

razzius commented 4 years ago

Good point @jjchen20. Currently editing through the admin will not cause the date updated to change but if you click "edit profile as admin" it will. I can update it to not update the last updated date in both of these cases.

jjchen20 commented 4 years ago

Ready to deploy. It looks like "edit profile as admin" still changes the last updated date.

razzius commented 4 years ago

This is now fixed so that admin edits will not update the last updated date.