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

hidden-desc-fix #2285

Merged
merged 3 commits into from
Dec 10, 2024
Merged

Conversation

Alwayswannahunt
Copy link

@Alwayswannahunt Alwayswannahunt commented Nov 30, 2024

Описание PR

Заметил странное несоответствие задумке и названиям, порешал.
Теперь скрытое описание находится на импланте, а не на имплантере, поэтому оно работает корректно. Имплантеры получили компоненты чтобы пушить описание того, что внутри. Корвакс отказался от этого, поэтому сразу к нам, лицензия по той же причине отсуствует.
Собрал наши импланты в одно место
Общедоступные импланты получили бирки, менее распространённые известны только тем, кто их использует.

Проверки

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

Изменения
🆑 AlwyAnri

  • fix: Исправлено отображение скрытого описания имплантера! Теперь оно зависит от импланта внутри имплантера!
  • fix: Общедоступные импланты получили бирки!

@github-actions github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: C# Изменение затронуло файлы ".cs" labels Nov 30, 2024
Copy link

github-actions bot commented Dec 1, 2024

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

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Dec 1, 2024
@github-actions github-actions bot removed the Merge Conflict Some content has conflicts. label Dec 2, 2024
@Alwayswannahunt
Copy link
Author

@Kirus59 Рестартни пж пж пж

@stalengd stalengd added the Need to be discussed Требуется обсуждение перед аппрувом/мёрджем label Dec 2, 2024
@stalengd stalengd requested a review from Ady4ik December 2, 2024 23:13
@UrPrice UrPrice removed the request for review from Ady4ik December 8, 2024 14:20
@stalengd stalengd removed the Need to be discussed Требуется обсуждение перед аппрувом/мёрджем label Dec 8, 2024
Copy link
Collaborator

Choose a reason for hiding this comment

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

Почему корвакс?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Потому что скрытые описания - от Корвуха?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ладно

@stalengd stalengd merged commit 8f5915f into SerbiaStrong-220:master Dec 10, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants