bcgov / cas-ciip-portal

The Climate Action Secretariat's CleanBC Industrial Incentive Program application
https://ciip.gov.bc.ca/
Apache License 2.0
10 stars 2 forks source link

CIIP Production view in metabase is missing values for Product Name #2192

Closed dleard closed 1 year ago

dleard commented 1 year ago

Many rows in the CIIP Production view in metabase are missing the Product Name field. This is because, when selecting a product as a reporter, the name is not saved to the form_result and the CIIP Production view pulls the (non-existent) product name from that form_result data.

We always have the product_id saved in the form_result, so having the name saved there too is redundant. The fix here is:

Probability (1-5): 4 The view will have blank product_name fields for any products selected via the dropdown by the reporter

Effect (1-5): 2 There is a work-around in metabase that has been shared in CIIP Internal

Steps to reproduce the behaviour:

dleard commented 1 year ago

@pbastia The bug occurred when a product was reported by selecting it from the dropdown. So to test this fix:

pbastia commented 1 year ago

@dleard I went through this, and the product_name field is empty in MB...maybe we can have a look together to see if I did something wrong

dleard commented 1 year ago

oops, looks like the deployment got stuck on a happo ci failure in shipit, so the fix didn't make it to dev. Deploying now

pbastia commented 1 year ago

I can see the product name in metabase. Thank you @dleard !

pbastia commented 1 year ago

I can see the product name in metabase. Thank you @dleard !