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

disease #218

Closed
wants to merge 27 commits into from
Closed

disease #218

wants to merge 27 commits into from

Conversation

Darkiich
Copy link
Contributor

Описание PR

Медиа

Проверки

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

Изменения

🆑

  • add: Добавлены болезни

@github-actions github-actions bot added Changes: UI Изменение интерфейса Changes: Localization Изменение локализации labels Aug 10, 2024
Copy link
Collaborator

@FaDeOkno FaDeOkno left a comment

Choose a reason for hiding this comment

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

Убирай ReagentEffect'ы и, желательно, апгрейды машин. Этот пр про болезни.
Пока что это всё

@Bolper Bolper added the Status: Awaiting Changes Ожидание изменений label Aug 19, 2024
Copy link
Contributor

@Bolper Bolper left a comment

Choose a reason for hiding this comment

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

Как кажется, тут должно быть великое перенаселение народов в папку ADT

@FaDeOkno
Copy link
Collaborator

Как кажется, тут должно быть великое перенаселение народов в папку ADT

Тут много чего нужно...

Copy link
Contributor

github-actions bot commented Sep 4, 2024

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

@Bolper
Copy link
Contributor

Bolper commented Sep 5, 2024

вопль ужаса
Screenshot_2024-09-05-21-38-35-022_com.github.android.png

Copy link
Contributor

github-actions bot commented Sep 6, 2024

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

Comment on lines +114 to +115
if (uid.Comp.Disease != null)
EntityManager.System<DiseaseSystem>().TryAddDisease(args.Target.Value, _proto.Index(uid.Comp.Disease.Value));
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
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение локализации Changes: UI Изменение интерфейса Merge Conflict Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants