Update moped_workgroup to reflect reorganization #1515
Conversation
mddilley
requested review from
frankhereford,
mateoclarke,
chiaberry,
johnclary,
tillyw and
roseeichelmann
| @@ -0,0 +1,3 @@ | |||
| -- Updating moped_workgroup table will be up only. If we need to revert, we will need do it manually or | |||
| -- update with a future migration. | |||
| SELECT 0; | |||
There was a problem hiding this comment.
I'm wondering if this is what I should've done on my last update to the moped_fund_status table.
There was a problem hiding this comment.
I like what you've got, Mateo. It reverts back to the previous DB state cleanly ✅ 🚢
There was a problem hiding this comment.
+1 - i like the down migration for the funding descriptions too. i've been omitting down migrations for these reorg changes since they update records in other tables and would take some heavier lifting to go back.
| SET | ||
| workgroup_id = (SELECT id FROM new_workgroup_row) | ||
| WHERE | ||
| workgroup_id IN (SELECT ids FROM user_todos); |
There was a problem hiding this comment.
pretty slick way of doing this—nice!
| moped_workgroup( | ||
| order_by: { workgroup_name: asc } | ||
| where: { is_deleted: { _eq: false } } | ||
| ) { |
There was a problem hiding this comment.
looks great! i did a quick scan of instances of moped_workgroup in the editor and couldn't find anything else that would be affected by this 👍
There was a problem hiding this comment.
thanks for checking! 🙏
Associated issues
Closes cityofaustin/atd-data-tech#19854
This PR updates workgroups with Sidewalks and Urban Trails and updates any users with Sidewalks or Urban Trails as their workgroup to fall under the new workgroup.
Testing
URL to test:
Local
Steps to test:
moped_workgrouptable and see that:Moped Test Adminin our vault or your own account) so you can see the staff form in later steps and then go to the Staff tabSidewalksand you should only see users withSidewalks and Urban TrailsworkgroupsSidewalks and Urban Trailsis available to selectShip list
[ ] Product manager added to QA test script if applicable