Bugfix news query #579
Bugfix news query #579
Conversation
списки год и месяц, теперь зависят друг от друга, это убирает проблему когда без выбора года нельзя было выбрать месяц
| })), | ||
| ], [year]); | ||
| const yearOptions = useMemo(() => { | ||
| let yearsArray: {text: string; value: string }[] = []; |
There was a problem hiding this comment.
можно вынести в утилитки маппинги
const getFilteredYearOptions = (filters, month, year) => {
const getFilteredMonthOptions = (filters, year) => {
There was a problem hiding this comment.
касательно этого, можешь подсказать вынести прямо в отдельную директорию или выше за компонент?
There was a problem hiding this comment.
@AlMkin @rkostyuchenko, парни, подскажите как лучше поступить в этой ситуации
There was a problem hiding this comment.
Тут дело вкуса. На мой взгляд, достаточно вынести за компонент.
Вообще, фильтр по месяцу без выбора года выглядит довольно бессмысленно: вряд ли кому-то понадобится найти новости за март каждого года. Возможно, лучше запретить пользователю выбирать месяц, пока не выбран год.
There was a problem hiding this comment.
Согласен, давайте блокировать выбор по месяцу без установленного года @MADeit0
There was a problem hiding this comment.
Погодите, но изначально же так и было, если год не выбран месяц не получится поставить, и соответственно после удаления года стирался и месяц.
There was a problem hiding this comment.
Выпадающий список с месяцами лучше явно задизейблить, пока не выбран год
Описание
Pull Request исправляет фильтрацию поиска по месяцу и году, теперь они зависимы друг от друга. Это значит, что если не выбран год или месяц в селектах будут отображаться все доступные варианты, но если выбрать конкретный год или месяц, то у второго селекта будут доступны только те варианты которые есть в рамка выбранного первым селектом.
Ссылка на задачу
Некорректно работающий фильтр по месяцам #560