j-palindrome / obsidian-time-ruler

A drag-and-drop time ruler combining the best of a task list and a calendar view (integrates with Tasks, Full Calendar, and Dataview).
MIT License
225 stars 8 forks source link

Usage of length DV task field conflicts with TS string length property #101

Closed stoilis closed 7 months ago

stoilis commented 7 months ago

Is your feature request related to a problem? Please describe. Time Ruler currently supports "length" as a task value to signify how long it lasts, if it doesn't have an endTime. However, length is a reserved Typescript word. When I use dataviewjs for advanced queries in obsidian, the conflict becomes more obvious. For example, this works:

But this doesn't:

Describe the solution you'd like I recommend changing the "length" to something like "effort" or "duration". The duration term is already a DataView datatype but it should not be an issue, because the function name is "dur()". If you want to avoid yet another potential conflict, "effort" seems like a better bet. Some people seem to already by using it this way.

j-palindrome commented 7 months ago

That's true and a good point. I can add support for parsing "duration" as well.