search

ut-effectiveness / utValidateR

Validate Data Elements in Warehouse and Reporting Pipelines
Other
0 stars 0 forks source link

C21a - Invalid Room Max Occupancy #95

Closed andreabringhurst closed 3 days ago

andreabringhurst commented 1 year ago

Add building abbreviation to the display.

cdemkegit commented 1 week ago

@mkhassan-isu This is one of the issues I hope may be resolved in the DataStewardsAudits

Steps to take:

  1. We can check to see if the data point is in the sql and/or pin
  2. We need to determine if this is something we can change in the App or do we need to change something in the utValidateR package as well.
  3. We can see what it takes to add a data point to the correct data table, this should be in the R/load_data.R file.
  4. We need to see what it takes to add that new field on the display tables in the Golem App itself, not sure what file that is.
andreabringhurst commented 1 week ago

I agree, they should be able to be addressed in the DataStewardsAudits since I'm requesting changes to the data displayed, not the data being checked.

mkhassan-isu commented 3 days ago

C29a and C37a have the same issue. Added the following: C21a- "meet_building_id_1", C29a- "meet_building_id_2" and C37a- "meet_building_id_3".