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

Добавил возможность вешать значки на броню #608

Conversation

Mirokko
Copy link
Contributor

@Mirokko Mirokko commented Oct 5, 2024

Описание PR

Название

Почему / Баланс

Ссылка на публикацию в Discord

Медиа

изображение
изображение
изображение
изображение
изображение
изображение

Требования

  • Я прочитал(а) и следую Руководство по созданию пулл реквестов. Я понимаю, что в противном случае мой ПР может быть закрыт по усмотрению мейнтейнера.
  • Я добавил скриншоты/видео к этому пулл реквесту, демонстрирующие его изменения в игре, или этот пулл реквест не требует демонстрации в игре

Чейнджлог
🆑

  • add: Добавлена возможность закреплять значок на броне СБ

Copy link
Contributor

github-actions bot commented Oct 5, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the Changes: Localization Изменение локализации label Oct 5, 2024
@Mirokko Mirokko force-pushed the feature/sec-badge-attachment-to-vests-and-coats branch from c1f1e1a to 5f3a7d3 Compare October 5, 2024 13:41
@Mirokko
Copy link
Contributor Author

Mirokko commented Oct 6, 2024

В общем, мне нужна помощь. Если подскажете, почему тесты упали, будет хорошо (и желательно хоть приблизительно как исправить).

@Mirokko Mirokko marked this pull request as draft October 6, 2024 06:10
@Mirokko Mirokko requested a review from Schrodinger71 October 6, 2024 06:10
@Mirokko Mirokko requested a review from KashRas2 October 7, 2024 06:27
@Mirokko
Copy link
Contributor Author

Mirokko commented Oct 7, 2024

Вроде тебя имели ввиду, нужно твоё мнение. Ща конфликт поправлю.

Copy link
Contributor

github-actions bot commented Oct 7, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Mirokko Mirokko force-pushed the feature/sec-badge-attachment-to-vests-and-coats branch from 5f3a7d3 to 2ed24ac Compare October 7, 2024 09:06
@Mirokko Mirokko marked this pull request as ready for review October 7, 2024 09:10
@Mirokko Mirokko force-pushed the feature/sec-badge-attachment-to-vests-and-coats branch from 77efe0b to 6b2adfa Compare October 7, 2024 13:32
@Mirokko
Copy link
Contributor Author

Mirokko commented Oct 9, 2024

@FaDeOkno возможно ты сможешь обьяснить что именно тесты хотят от меня, я не понимаю. Есть только примерное понимание проблемы

@Darkiich
Copy link
Contributor

@Mirokko На такие тесты с нулевой локалью забивай болт, почему? Потому что это база у нас и так сказал Котя

Copy link
Contributor

@PyotrIgn PyotrIgn left a comment

Choose a reason for hiding this comment

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

Добавь в комментарии к пэренту вне АДТшных папок - что это наше улучшение

@Mirokko Mirokko requested a review from PyotrIgn October 29, 2024 10:23
@PyotrIgn PyotrIgn added this pull request to the merge queue Oct 30, 2024
Merged via the queue into AdventureTimeSS14:master with commit b59c69f Oct 30, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение локализации Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants