s0lvang / Karantenehjelpen

A web-application for helping people in quarantine to get groceries
https://karantenehjelpen.no
27 stars 6 forks source link

Bruk date-fns for å formattere tid #147

Closed FredrikAugust closed 4 years ago

FredrikAugust commented 4 years ago

Ser litt mer menneskelig ut med "2 timer siden" kontra "Torsdag 19. mars 15:00".

Screenshot 2020-03-19 at 17 00 33
oleast commented 4 years ago

Hva med native API-er i stedet for moment? https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RelativeTimeFormat

Adrianht commented 4 years ago

https://github.com/you-dont-need/You-Dont-Need-Momentjs

FredrikAugust commented 4 years ago

NativeAPI hadde vært nice, men ser ikke ut som om det har så god støtte

https://caniuse.com/#search=relativetime

FredrikAugust commented 4 years ago

date-fns ser ut som om det gjør samme nytta. Brukte bare moment fordi det er det jeg pleier å bruke, og er superlett å bruke

Skal jeg bruke tid på å bytte det over til date-fns?

oleast commented 4 years ago

NativeAPI hadde vært nice, men ser ikke ut som om det har så god støtte

https://caniuse.com/#search=relativetime

Er ikke så god støtte, men det skal finnes polyfills.

Ellers vil jeg si date-fns eller Luxon er gode alternativer som ikke er likke brukket som moment

FredrikAugust commented 4 years ago

Ser folk snakker om at moment er brukket, men har aldri hatt noen problemer med det selv. Hva er problemet egt?

FredrikAugust commented 4 years ago

// luxon ❌ does not support relative time

FredrikAugust commented 4 years ago

Du må gjerne oppdatere PRen hvis du har innsikt i hvordan det kan gjøres bedre.

Adrianht commented 4 years ago

Har hatt litt funky problemer med det selv før, men det er jo ikke noe vi kommer til å bruke mye i den siden her (probably). Eneste jeg kan tenke på her er at det er en større pakke enn f.eks date-fns

FredrikAugust commented 4 years ago

@Adrianht @oleast bedre?