search

pacificclimate / weather-anomaly-tool

0 stars 0 forks source link

Set default date to latest with data #55

Closed rod-glover closed 4 years ago

rod-glover commented 4 years ago

Resolves #52

faronium commented 4 years ago

Thanks for getting to this guys! Have a good long weekend

rod-glover commented 4 years ago

@jameshiebert , I've replaced the date arithmetic with the moment package, which AFAIK is still the current best date package, despite its habit of mutating its values. Code is tighter.

rod-glover commented 4 years ago

Demo. As they say in ads for cleaning products, "you can see it working."

rod-glover commented 4 years ago

@jameshiebert , Demo updated. Quick eyeball?

FYI, I verified that it is requesting the expected months. The backend expects 1-based months, hence Mar == 3, for example.

jameshiebert commented 4 years ago

Yeah, looks good!