Skip to content
This repository has been archived by the owner on Oct 15, 2022. It is now read-only.

bbc - using moment.js, removed unnecessary helpers #1910

Merged
merged 1 commit into from Jun 15, 2015

Conversation

iambibhas
Copy link
Collaborator

Saw this IA could use some moment.js love, so using that. Sadly can't remove the duration() function as moment.js' humanize() function doesn't return accurate duration.

@moollaza
Copy link
Member

Sadly can't remove the duration() function as moment.js' humanize() function doesn't return accurate duration.

Care to elaborate on that? Is it a bug in Moment.js?

@iambibhas
Copy link
Collaborator Author

Is it a bug in Moment.js?

See here - moment/moment#348

So when the show duration is 3 hours 15 minutes, the .humanize() function of moment.js returns 3 hours and totally ignores the 15 minutes.

@moollaza
Copy link
Member

@iambibhas can we do what was described in moment/moment#348 (comment)?

I defined a modification to the language in a similar fashion for the Tor Node IA because of a similar problem.

@moollaza
Copy link
Member

I guess its about as much code either way. This is fine for now 👍

moollaza added a commit that referenced this pull request Jun 15, 2015
bbc - using moment.js, removed unnecessary helpers
@moollaza moollaza merged commit ace76c5 into duckduckgo:master Jun 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants