adopted-ember-addons / ember-moment

MIT License
399 stars 122 forks source link

interval not recomputing properly with (now) #248

Closed RobbieTheWagner closed 7 years ago

RobbieTheWagner commented 7 years ago

I tried using the now helper and it never recomputes:

{{moment-format (now) 'LT' interval=1000}}

However, if I pass undefined, it does recompute:

{{moment-format undefined 'LT' interval=1000}}

Any ideas on this? Would like to be able to pass (now) instead of undefined.

jasonmit commented 7 years ago

I believe you want {{moment-format (now interval=1000) 'LT'}}

jasonmit commented 7 years ago

Feel free to reopen if this continues to be an issue

RobbieTheWagner commented 7 years ago

@jasonmit shouldn't we update the README then? It doesn't show that here https://github.com/stefanpenner/ember-moment#live-updating-of-displayed-time

jasonmit commented 7 years ago

@rwwagner90 thanks for calling that out - I've updated accordingly and sorry the docs mislead you!