natural-time-delta sorting - remove the splitter key and split by full keyword

[saveshodhan]

• Removed unwanted data
• Corrected 'splitter' values passed for calculating time

[saveshodhan]

JSFiddle for this update:
https://jsfiddle.net/9vvd4fvr/1/

[DataTables]

Does that mean that you have to put the full text rather than just "1h 4m" for example?

[saveshodhan]

I guess that was always the case.. Because the regular expression expects either hour or hours.. Reason was that 'hour' could be represented as h or hr or hour (likewise for plural cases). so i'd thought of matching common keywords (like hours or minutes, etc.)

In this commit, i have simply removed the splitter key (as it seemed redundant) and am sending the name itself as splitter..

We may go ahead with this change for now, i will have to think of how to implement shorter time formats as well..

What say?

[saveshodhan]

Hello,

Hope you are doing well..!

I was going through the repo, and saw this commit waiting to merge.
I see a minor typo from my end that I am going to fix.

Meanwhile, regarding the query that you had asked in a comment above, just wanted to make sure that i was able to answer it in the subsequent comment, and that we are on the same page..

[saveshodhan]

Probably what I meant above was that the string 2 months was earlier split using m, then I changed it to be split by month (JS would still give me the number 2 on 0th index). This is done because of an error - in the mappings, we have splitter as w for the last three elements (I guess "Refactoring" is a misleading name, will change it). So, if splitting by month or m is going to give same result, why have an extra splitter element in the dict?

Also, there is this ambiguity that m might mean months or minutes.

This ambiguity is also why I had decided to stick to full text rather than having short forms, which may answer your query

Does that mean that you have to put the full text rather than just "1h 4m" for example?