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

Cult Yogg cultists count major trigger #2634

Merged
merged 2 commits into from
Feb 26, 2025

Conversation

stalengd
Copy link
Collaborator

@stalengd stalengd commented Feb 25, 2025

Описание PR

По задаче

Культисты с горящими зелёными глазами (Можно скрыть очками/маской) - Проявление после принесения первой жертвы или при достижении культом численности в 20% или 10% от количества экипажа (зависит от количества экипажа при старте раунда)

Культисты с коричнево-голубым нимбом над головой - Проявление после принесения второй жертвы при достижении культом численности в 30% или 20% от количества экипажа (зависит от количества экипажа при старте раунда)

Медиа

-

Проверки

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

Изменения

🆑

  • tweak: Теперь культ Йог-Сотот становится мажорным при достижении определённого количества последователей

@github-actions github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: C# Изменение затронуло файлы ".cs" Status: Needs Review labels Feb 25, 2025
@stalengd stalengd added the Need to add Wiki220 Требуется добавить на Wiki/поставить задачу Wikiделам вместе с мёрджем. label Feb 25, 2025
@UrPrice UrPrice removed the request for review from SkaldetSkaeg February 25, 2025 22:04
@Tris0r
Copy link

Tris0r commented Feb 26, 2025

b861979812a203bbe2f1d7e84060a52b.mp4


[DataField]
public int AmountOfSacrificesToWarningAnouncement = 2;
public int ReqAmountOfMiGo = 3;
Copy link

@SkaldetSkaeg SkaldetSkaeg Feb 26, 2025

Choose a reason for hiding this comment

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

что за необходимое число Ми-Го? Я что забыл это удалить? ебббт...

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ну типа того, там по крайней мере несколько полей я видел не используется

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.

ну, это одно из них, которое можно смело удалить

Чистку кода лучше провести в отдельном ПРе

@@ -332,40 +333,51 @@ private void AfterEntitySelected(Entity<CultYoggRuleComponent> ent, ref AfterAnt
MakeCultist(args.EntityUid, ent);
}

public void MakeCultist(EntityUid uid, CultYoggRuleComponent comp, bool initial = true)
public void MakeCultist(EntityUid uid, Entity<CultYoggRuleComponent> rule, bool initial = true)

Choose a reason for hiding this comment

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

хз, зачем сюда энтити геймрула передавать

Choose a reason for hiding this comment

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

только из-за TryGetNextStage?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Я решил везде передавать компонент вместе с его энтити, потому что иначе возникнет неприятная ситуация что где-то в глубине этот энтити в итоге пригодится, а его нет

Choose a reason for hiding this comment

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

не находил не единой ситуации, где идет работа именно с энтити геймрула, но хз, может оффы что добавят

@Ady4ik
Copy link

Ady4ik commented Feb 26, 2025

@Burenov need to be add on wiki

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 26, 2025

@Burenov need to be add on wiki

I did it yet

@UrPrice UrPrice removed the Need to add Wiki220 Требуется добавить на Wiki/поставить задачу Wikiделам вместе с мёрджем. label Feb 26, 2025
@Kirus59 Kirus59 merged commit 9a3c612 into SerbiaStrong-220:master Feb 26, 2025
37 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"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants