Skip to content

feat(components/input-date-time): added native date transformation for min max #1315#1683

Merged
ZurabDev merged 26 commits intov4from
fix/min-max-date-transformer-1315
Sep 23, 2024
Merged

feat(components/input-date-time): added native date transformation for min max #1315#1683
ZurabDev merged 26 commits intov4from
fix/min-max-date-transformer-1315

Conversation

@ickisIckis
Copy link
Copy Markdown
Contributor

@ickisIckis ickisIckis commented May 14, 2024

feat(components/input-dates): added native date transformation for min max properties #1315
feat(components/input-dates): added date string transformation for min max properties #1573
feat(components/cron): added native date transformation for min max properties in date inputs #1315
feat(components/input-layout-date): added new provider for ISO and UTC strings transformer for input layout date #1574
feat(doc): remove examples for deleted prizm-input-date component

Библиотека

  • @prizm-ui/components
  • @prizm-ui/install
  • @prizm-ui/icons
  • @prizm-ui/theme

Компонент

контролы для выбора дат

Задача

  • Ссылка на issue(если есть)

Изменения

  • Имеются BREAKING CHANGES
  • Изменения документации
  • Добавление фичи
  • Исправление бага

Checklist:

  • После фичи обновил документацию
  • Сделал код чище чем был до этого
  • Тесты и линтер на рабочей машине успешно выполнились

Следует обратить внимание на ревью

Все контролы дат компоненты их содержащие, особенно крон. live deomo, новые примеры.
resolved #1315
resolved #1573
resolved #1574

@ickisIckis ickisIckis added this to the 4.3.0 milestone May 14, 2024
@ickisIckis ickisIckis added the state:in_progress We work on this issue label May 14, 2024
@ickisIckis ickisIckis modified the milestones: 4.3.0, 4.4.0 May 14, 2024
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2024

Visit the preview URL for this PR (updated for commit 17c1816):

https://prizm-v4--pr1683-fix-min-max-date-tra-t03w3wlc.web.app

(expires Wed, 28 Aug 2024 05:09:14 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 7c62ed8dbabf5e2d6b2084ca9e107cc206d30dbd

@ickisIckis ickisIckis force-pushed the fix/min-max-date-transformer-1315 branch from 522ba28 to d78be71 Compare July 7, 2024 22:46
@ickisIckis ickisIckis force-pushed the fix/min-max-date-transformer-1315 branch from cf1b0e9 to c44394a Compare July 30, 2024 09:11
@ickisIckis ickisIckis removed the state:in_progress We work on this issue label Jul 30, 2024
Copy link
Copy Markdown
Contributor

@ZurabDev ZurabDev left a comment

Choose a reason for hiding this comment

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

LGTM

@alexhawkins94
Copy link
Copy Markdown

С клавиатуры можно задать значение за границами разрешенного диапазона
Uploading chrome_rIy3mzCveM.png…
ОР: автоисправление на ближайшее граничное значение при ручном вводе как в примере inputdatetime min max with time

@alexhawkins94
Copy link
Copy Markdown

При выборе max = invalid string в календаре так же доступен 2027 год в качестве верхнего предела

При выборе max = {} в календаре доступен максимально 2027 год в качестве верхнего предела
если выбрать min и max = {} то недоступно для выбора ни одна дата в календаре

@ickisIckis
Copy link
Copy Markdown
Contributor Author

Автоисправление не было реализовано в этом контроле, будет сделано на отдельной ветке #1995

Copy link
Copy Markdown
Contributor

@ZurabDev ZurabDev left a comment

Choose a reason for hiding this comment

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

LGTM

@PrizmDS
Copy link
Copy Markdown
Contributor

PrizmDS commented Aug 26, 2024

Необходимо проверить сценарий: В случае невалидной строки в значении, необходимо очищать поле, а не возвращать предыдущее валидное значение.

#1994

@ickisIckis ickisIckis changed the base branch from main to v4 August 26, 2024 11:52
@VeronikaRpn
Copy link
Copy Markdown

При клике на Set invalid string - передается Invalid string вместо Null
image

@ickisIckis
Copy link
Copy Markdown
Contributor Author

При клике на Set invalid string - передается Invalid string вместо Null image

Приняла решение убрать пример с невалидной строкой из доки.
Мотивация:
пользователь не имеет возможности установить невалидную строку вручную, она может попасть туда только программно: либо прилетит с бека, либо что-то пойдет не так в приложении. В этом контексте считаю, что пробелма должна решаться на уровне кода проекта (не устанавливать в контрол даты не-дату), а с нашей разработчики должны быть информированы о проблеме - ошибка в консоли присутствует, контрол очищается. Но устанавливать новое значение с нашей стороны после отработки трансформера выглядит архитектурно неправильно.

@alexhawkins94
Copy link
Copy Markdown

В input date time range возможно задать время через дропдаун за пределами разрешенного диапазона в min-max.
chrome_mNHX260fOA

@ickisIckis
Copy link
Copy Markdown
Contributor Author

ickisIckis commented Sep 16, 2024

В input date time range возможно задать время через дропдаун за пределами разрешенного диапазона в min-max. chrome_mNHX260fOA

это новая функция, добавлена на другой ветке

@ZurabDev ZurabDev merged commit d7877d0 into v4 Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

5 participants