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

fixed appStatus translation #521

Merged
merged 3 commits into from
Jul 5, 2024
Merged

Conversation

bobrov-site
Copy link
Contributor

Исправлен перевод спиннера загрузки по умолчанию. Ранее был текст appStatus.loading, сейчас отображает корректные перевод на 2 языках.

Снимок экрана 2024-06-27 в 15 17 53

@bobrov-site
Copy link
Contributor Author

Попрошу тестировщиков внимательно проверить перевод спиннеров. Если будет так же указано appStatus.loading или не указан, то значит баг актуален.

Демо https://bobrov-runit.onrender.com

@bobrov-site
Copy link
Contributor Author

@KseniyaKerzhner можете пожалуйста посмотреть по демке?
@HelenOne можешь пожалуйста код ревью сделать?

@KseniyaKerzhner
Copy link

KseniyaKerzhner commented Jul 1, 2024

@bobrov-site И в Английской и в русской версии всё корректно работает, светлая и темная темы так же проверены всё хорошо.
Вопрос только по стилистике, "Loading" с троеточием и "Загрузка" нет. Правильно ли это?

2024-07-01_12-26-53
2024-07-01_12-30-42

@bobrov-site
Copy link
Contributor Author

@KseniyaKerzhner поправлю

@bobrov-site
Copy link
Contributor Author

Не могу понять почему тесты валятся. После регистрации не переходит на страницу с пользователем?

@KseniyaKerzhner
Copy link

У меня всё переходит, надпись поменялась, всё хорошо.

@bobrov-site
Copy link
Contributor Author

@KseniyaKerzhner не, я имел ввиду по playwright тестам.

@bobrov-site
Copy link
Contributor Author

Решил проблему с тестами. @HelenOne можно сливать в мастер.

@HelenOne HelenOne merged commit 92529a4 into hexlet-rus:main Jul 5, 2024
2 checks passed
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.

3 participants