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

ISSUE #150 Changed path for translator in app #510

Conversation

VictoryPashkova
Copy link
Contributor

Изменила путь до переводчика в компонентах с помощью keyPrefix
Пр. для постоянных ключей:
const { t: tF } = useTranslation('translation', { keyPrefix: 'footer' }); {tF('about')}

Для динамически вычисляемых ключей сохраняется подход:

const { t } = useTranslation();
{t(formState.message)}

@fey fey requested a review from HelenOne June 17, 2024 16:08
@fey
Copy link
Contributor

fey commented Jun 17, 2024

Посмотрите, пожалуйста, почему тесты упали.

@HelenOne
Copy link
Contributor

@VictoryPashkova надо тесты починить, чтобы мержить можно было

@VictoryPashkova
Copy link
Contributor Author

Починю тесты, да

@HelenOne
Copy link
Contributor

@VictoryPashkova опять тесты падают и новые конфликты появились

@VictoryPashkova
Copy link
Contributor Author

@HelenOne 2/3 я нашла и поправила, помоги пожалуйста с тестом Unable authorization by invalid password. Судя по выводу он не может перейти на какую то страницу если что-то не так с паролем await page.waitForURL(http://localhost:3000/u/test35); Но я же только тексты правила и не должна была на функциональность навигации влиять. Модуль с модальным окном регистрации я проверила, там по текстам все ок. Запусти проверку тестов снова, посмотрю что они покажут после исправления первых двух ошибок

@HelenOne
Copy link
Contributor

@VictoryPashkova сначала лучше конфликты решить. Там произошли изменения в Footer.jsx

@VictoryPashkova
Copy link
Contributor Author

@HelenOne все работает, конфликты разрешила, тесты прошли

@HelenOne HelenOne merged commit 0bc147b into hexlet-rus:main Jun 26, 2024
2 checks passed
@bobrov-site
Copy link
Contributor

@VictoryPashkova можете пожалуйста убрать строчку ниже, так как линтер выбрасывает предупреждение. Я просто увидел, что вы вносили изменение в файл в этом пулл реквесте.

const { t: tFAQ } = useTranslation('translation', { keyPrefix: 'faq' });

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.

5 participants