cityofaustin / atd-moped

A comprehensive mobility project tracking platform for Austin, Texas.
https://mobility.austin.gov/moped/
4 stars 3 forks source link

fixes bug where role update is reflected as notes update #1424

Open tillyw opened 3 days ago

tillyw commented 3 days ago

Associated issues

https://github.com/cityofaustin/atd-data-tech/issues/16950

this issue stems from the moped_proj_personnel_roles table not being associated with a project_id (rendering that field as null) and therefore not returning records as part of the moped_activity_log query. investigating this has uncovered some potential tech debt when it comes to the activity log triggers (as noted in this pr), but @mddilley and i decided in the short term to fix the activity log rendering incorrect associations in the ui by assuming that a change made to the moped_proj_personnel table that is not related to a user or note is related to a role.

Testing

URL to test:

https://deploy-preview-1424--atd-moped-main.netlify.app/

Steps to test:


Ship list