Skip to content

Conversation

@oburdasov
Copy link
Contributor

@lskramarov Добавил объект options с seconds и milliseconds в аргументы всех методов, оканчивающихся на DateTime.

Я думаю, что в доках нет смысла добавлять секунды и миллисекунды во все форматы, получится слишком громоздко.

Лучше сделать снизу отдельный раздел, и в нём написать, что все DateTime методы принимают DateTimeOptions

@lskramarov
Copy link
Contributor

обрати внимание, что разделитель миллисекунд для ru и en отличается:
image

yes{{SHORT_DATE}, {TIME}}
other{{SHORT_DATE}, {YEAR}, {TIME}}
}{
SHOW_MILLISECONDS,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

выше написал, это нужно будет вернуть.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

вернул

@lskramarov
Copy link
Contributor

Я думаю, что в доках нет смысла добавлять секунды и миллисекунды во все форматы, получится слишком громоздко.

доками пользуются не только разработчики, поэтому нужно показать все варианты (в гайде так же полные примеры).

@oburdasov
Copy link
Contributor Author

@lskramarov переделал + добавил примеры и тесты. Можно смотреть.

Единственное, что для открытых диапазонов вообще не было примеров.
Если нужны, то предлагаю создать на это отдельную задачу, т.к. этот pr и так уже огромный

@oburdasov oburdasov marked this pull request as ready for review February 1, 2022 21:56
@lskramarov lskramarov merged commit 690611a into positive-js:master Feb 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants