Sorry for not reviewing earlier... Might it be possible that you write a test for this?

Sorry for not reviewing earlier... Might it be possible that you write a test for this?

I am willing, but in all honesty I have never written an automated test so there is going to be a bit of a learning curve. If I could be directed to something existing I can base it off of that would be appreciated.

Sorry for not reviewing earlier... Might it be possible that you write a test for this?

I am willing, but in all honesty I have never written an automated test so there is going to be a bit of a learning curve. If I could be directed to something existing I can base it off of that would be appreciated.

You can take a look at tests/e2e/modules/weather_hourly_spec.js
There are tests for checking the precipitation is displayed. Your test should check if the 0 is displayed if the pop in the mocked data tests/mocks/weather_hourly.json is 0

Merging this so the next release has it. If you find the time @vppencilsharpener you can always add a test later ;-)

Thanks for your job but since this change, the amount of precipitaton (quantity of water) seems to be 0.00 mm and for me it's not relevant. When your return null, the column could disappaear if we have a week without rain !
Right now, i see a value for all days. Can we make a revert or a better fix ?