Closed bitvijays closed 3 years ago
@anshulgaikwad01 Thank for the good work. Can you ensure lines https://github.com/bitvijays/AirPollutionDataPlatform/pull/30/files#diff-22b192b9f2aabd6d42918f4d64ebbe1d07062ce89d79388441dcee4c8459c043L803 are good and in sync?
Also, may instead of using toFixed(2) two times. Maybe we should have it like the original author did here https://github.com/bitvijays/AirPollutionDataPlatform/blob/master/website/public/js/map.js#L602
toFixed(2)
i.e create a variable valueToDisplay and check for NaN and then use that ? for consistency.
valueToDisplay
NaN
Thank You @anshulgaikwad01 for your efforts! Merged
@anshulgaikwad01 Thank for the good work. Can you ensure lines https://github.com/bitvijays/AirPollutionDataPlatform/pull/30/files#diff-22b192b9f2aabd6d42918f4d64ebbe1d07062ce89d79388441dcee4c8459c043L803 are good and in sync?
Also, may instead of using
toFixed(2)
two times. Maybe we should have it like the original author did here https://github.com/bitvijays/AirPollutionDataPlatform/blob/master/website/public/js/map.js#L602i.e create a variable
valueToDisplay
and check forNaN
and then use that ? for consistency.