search

positive-js / mosaic

Angular components ✨ 📦🐣 ✨
https://mosaic.ptsecurity.com
MIT License
142 stars 40 forks source link

feat(formatter): add seconds and milliseconds options (#UIM-781) #793

Closed oburdasov closed 2 years ago

oburdasov commented 2 years ago

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

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

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

lskramarov commented 2 years ago

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

lskramarov commented 2 years ago

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

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

oburdasov commented 2 years ago

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

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