Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix] Added a check if recipient isUnConfirmed in requesting complete… #265

Merged
merged 3 commits into from
Jul 30, 2024

Conversation

KuzmichevaES
Copy link
Contributor

При работе с неподтвержденным реципиентом добавлено исключение запросов на сервер за завершенными заявками.

Copy link
Collaborator

@INextYP INextYP left a comment

Choose a reason for hiding this comment

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

Было бы здорово, если бы и вот эти запросы не уходили, если реципиент неподтвержден...

image

@KuzmichevaES
Copy link
Contributor Author

Добавила skip, чтобы не уходил запрос.

Copy link
Collaborator

@INextYP INextYP left a comment

Choose a reason for hiding this comment

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

Запрос всё также уходит :)

Обрати внимание, как работает skip.
Он пропустит запрос, если условие равно true.

Если детально посмотреть процесс записи в isUnConfirmedSelector селектор данных, то можно увидеть, что в селектор сначала запишется false (из-за этого и полетит запрос), а только потом true

Условие нужно дополнить. Добавить (как вариант) проверку, что пока у нас нет пользователя И статус неподтвержден - запрос делать мы не будем

@KuzmichevaES
Copy link
Contributor Author

Добавила проверку User, но т.к. запросы не уходили, выполнялось условие !data и рендерился loader. Не знаю, насколько это корректное решение, но убрала проверку data, создала новую переменную, в которой или data или пустой массив, чтобы карта отрисовывалась даже если не ушел запрос и пока не пришла data.

Copy link
Collaborator

@INextYP INextYP left a comment

Choose a reason for hiding this comment

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

Отлично. Задача выполнена!

@INextYP INextYP merged commit ecc30c1 into develop Jul 30, 2024
@INextYP INextYP deleted the fix/unconfirmed-recipient branch July 30, 2024 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Исправления в работе с неподтвержденным реципиентом
2 participants