Некорректная работа роутинга about react

[Daria-Kuzminykh]

Промежуточный результат:

1. Убрала логику с rememberedPrevRouterParamsRef, потому что такая же логика работает внутри usePrevious.
2. Основная проблема: prevRouteParams в какой то момент обновляются лишний раз, поэтому становятся равными routeParams.
3. Для сохранения параметров использовала usePreviousDistinct вместо usePrevious. Это решило проблему с вариантами 3, 7, 8. В вариантах 4, 5 проблема осталась. Вариант 6 не смогла воспроизвести.

Еще показалось, что нужно вот такое изменение в проекте:

Но оно не решило никакой проблемы. Но и не сломало ничего. Его я не сохраняла

Все изменения сохранила в репо react ветке routing-bug

Варианты дальнейшего решения:

1. Написать свой usePrevious, который будет нормально работать на всех вариантах
2. Переписать логику роутинга модалок как нибудь по другому

[Daria-Kuzminykh]

Итог:

Кейсы 3, 7, 8 пофикшены в этом pr
Кейс 5 пофикшен в репо проекта Край родной в ветке update_steroids (см. последний коммит)
Кейсы 4, 6 работают некорректно из-за редиректа на страницу not-found в проекте Край родной в файле AudioTextModal.tsx (оставила там туду, как можно пофиксить эту проблему)

@devmartynov Дальнейшие шаги:

1. Поставила тебя ревьювером в этом pr, посмотри изменения, прокомментируй если нужно
2. Смержить pr с исправлениями в репо react и поднять версию
3. Пофиксить проблему с редиректом на страницу not-found в проекте

[devmartynov]

На каких тест-кейсах роутинг работает некорректно и в чем заключается некорректность?

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

[perlexed]

На каких тест-кейсах роутинг работает некорректно и в чем заключается некорректность?

[perlexed]

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

Кто сможет этим заняться? @devmartynov @Daria-Kuzminykh @Dmitriy-Babiy ?

[devmartynov]

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

Кто сможет этим заняться? @devmartynov @Daria-Kuzminykh @Dmitriy-Babiy ?

я займусь

[devmartynov]

@perlexed

я займусь

Примеры некорректной работы

3. /book/kray-rodnoy/chapter_2 --> /book/kray-rodnoy/chapter_2/dictionary/1 + x = /book/kray-rodnoy/chapter_2?articleId=1
4. book/kray-rodnoy/chapter_2 --> /book/kray-rodnoy/chapter_2/audio-text?audioId=2 + НН = not found
5. /book/kray-rodnoy/chapter_2 --> /book/kray-rodnoy/chapter_2/audio-text?audioId=2 + x = /book/kray-rodnoy/chapter_2?audioId=2
6. /book/kray-rodnoy/chapter_3?test=3 —> /book/kray-rodnoy/chapter_2/audio-text?audioId=3 + НН = not found
7. /book/kray-rodnoy/dictionary?chapterId=5 —> /book/kray-rodnoy/dictionary/9 + x = /book/kray-rodnoy/dictionary?articleId=9
8. /book/kray-rodnoy/dictionary/1 + x = /book/kray-rodnoy/dictionary?articleId=1

not found - это статус из хука useLayout

[perlexed]

@devmartynov это для информации? или от меня по задаче что-либо нужно?

[devmartynov]

@perlexed

Я неправильно понял вопрос

Кто сможет этим заняться?

Отнес этот вопрос к тому, что надо проверить тесты на исправленном коде. Поэтому надо снова людей пингануть с этим вопросом:)

Кто может заняться решением проблемы? @Daria-Kuzminykh @Dmitriy-Babiy

[devmartynov]

@Daria-Kuzminykh добавился еще один кейс некорректной работы:

1. |--> МОК + x

Пример:

|--> /book/kray-rodnoy/chapter_2/audio-text?audioId=2+ x = /book/:bookName/:chapterId

2. Также есть вот такой момент:

ММ --> ММ

То есть это ситуация, когда пытаемся открыть два маршрута(role='modal') одновременно. В этом случае первое модальное окно закрывается.