[* DateTimeV2] Added support for patterns like "week one" (#2964)

Open aitelint opened this issue 3 years ago • 1 comments

Fix to issue #2964.

Added support for "week one" in EN and NL. Added support for "first week" in EN (where, if the year is missing, the start/end dates are calculated from the reference date).

Test cases added to EN and NL DateTimeModels.

Jun 08 '22 08:06 aitelint

But why add suppoer for ordinal + week w/o context in this way in English? Such cases without context need a different timex.

I left a comment in PR 2963 when we were talking about this pattern, but since there was no answer I followed this approach.

What timex should they have?

Jul 13 '22 12:07 aitelint