metrotranscom / doorway

component-based web framework for affordable housing management
Apache License 2.0
3 stars 1 forks source link

AMI NaN #692

Open sarahlazarich opened 2 weeks ago

sarahlazarich commented 2 weeks ago

Bug:

image.png Screenshot 2024-06-18 at 10.08.08 AM.png

Expected Behavior:

sarahlazarich commented 2 weeks ago

Should fix in Core as well

sarahlazarich commented 9 minutes ago

Addressing @mcgarrye's comment on the PR for this.

I spoke with Izzie, and we can instead require that units are saved with AMI charts if it that helps avoid the weird behavior @mcgarrye called out.

mcgarrye commented 7 minutes ago

Addressing @mcgarrye's comment on the PR for this.

I spoke with Izzie, and we can instead require that units are saved with AMI charts if it that helps avoid the weird behavior @mcgarrye called out.

That can definitely work but that doesn't fix all existing units without them and we wouldn't be able to backfill them