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

[UPDATE Xeno and litle FIX Chaplain] #912

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

NameLunar
Copy link
Contributor

@NameLunar NameLunar commented Dec 26, 2024

Описание PR

Обновление Ксеноморфов и мелкий фикс священника.

Техническая информация

Если прочитать код, то вроде все понятно.
В компоненте XenoQueen (Серверный) можно изменять некоторые параметры и отключать то или иное.
Очки королевы не показываются и только в компоненте можно посмотреть их количество.
Очки королевы показываются как иконка ниндзя костюма. (Типа цистерна крови. Не обязуйте). Максимум 150. Каждая полоска = 23 очкам.
Причиной этого является отсутствие спрайтов и идей другой кастомизации. Из плюсов можно выделить то, что недостающие очки пишутся при попытке вызвать кого либо.

Требования

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

Чейнджлог
🆑 NameLunar, при поддержке FaDeOkbo

  • add: Ксеноморфы вновь эволюционируют!
  • fix: Потерянный священник вновь верует в богов!
  • tweak: Теперь Королеве ксеноморфов требуются очки для спавна других ксенов. (Не получится призвать 999 мобов сразу😢)
  • tweak: Обновлена одна деталька у шаттла Службы безопасности!

@github-actions github-actions bot added size/L Changes: Localization Изменение локализации S: Untriaged labels Dec 26, 2024
Copy link
Contributor

@Unlumy Unlumy left a comment

Choose a reason for hiding this comment

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

В целом, практически всё, что я здесь увидел сделано правильно, так что можно будет мерджить если ты поправишь.
Однако .cs файлы бы глянуть кому-нибудь, например @FaDeOkno или @KashRas2

@Unlumy
Copy link
Contributor

Unlumy commented Dec 26, 2024

А, я не увидел, что драфт, но ты всё равно поправь.

@NameLunar
Copy link
Contributor Author

Я не понимаю. Никак не могу понять причину ошибки
[ERRO] proto: Reading Robust.Shared.Prototypes.EntityPrototype(ActionSpawnMobXenoBurrower) threw the following exception: System.Invalid Operation Exception: Type 'Content.Shared.Actions.WorldTargetActionEvent' is abstract, but could not find concrete type 'SpawnXenoQueenEvent'. at Robust.Shared.Serialization.Manager.SerializationManager.ResolveConcreteType(Type baseType, String typeName)

@COTE-LAPYX
Copy link

ПР конечно в драфте, но такие проблемы стоит убирать сразу, а лучше вообще выкинуть привычку все писать в одном пре.

Баг фикс со спавном священника не должен быть тут. Вы же даже ставите галочку что прочитали и следуете руководству по созданию ПРов...
Screenshot_2025-01-03-09-42-15-311_com.android.chrome-edit.jpg

Спасибо.

На счёт ошибки я не уверен до конца не чекал, но пересмотри где регестрируются ивенты на призыв, и сами ивенты.
Ещё скажи, работает ли сам спавн и запускается ли все вообще, либо же это просто проверка шалит.

@github-actions github-actions bot added the Changes: Map Изменение карт label Jan 3, 2025
@NameLunar
Copy link
Contributor Author

Да я уже все исправил. Ошибка была глупая... Про священника да. Но блин, почему и нет? Всего то 10 символов от силы

@NameLunar NameLunar changed the title [X] [UPDATE Xeno and litle FIX Chaplain] Jan 3, 2025
@NameLunar
Copy link
Contributor Author

Сейчас это можно использовать и играть, но проблема точным количеством очков...

@NameLunar NameLunar marked this pull request as ready for review January 3, 2025 14:32
@NameLunar NameLunar closed this Jan 3, 2025
@NameLunar NameLunar reopened this Jan 3, 2025
@NameLunar NameLunar closed this Jan 3, 2025
@NameLunar NameLunar reopened this Jan 3, 2025
@NameLunar NameLunar closed this Jan 3, 2025
@NameLunar NameLunar reopened this Jan 3, 2025
@NameLunar
Copy link
Contributor Author

Ну... Ладно. Займусь позже

@github-actions github-actions bot added the Changes: Sprites Изменение спрайтов label Jan 4, 2025
Copy link
Contributor

github-actions bot commented Jan 4, 2025

RSI Diff Bot; head commit a649e96 merging into e16143d
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/ADT/Interface/Alerts/XenoQueenBloobCount.rsi

State Old New Status
power0 Added
power1 Added
power2 Added
power3 Added
power4 Added
power5 Added
power6 Added
power7 Added

Edit: diff updated after a649e96

@NameLunar
Copy link
Contributor Author

И что это значит?
PoolManagerTestEventHandler: ERROR: Tests are taking too long. Shutting down all tests. This may lead to weird failures/exceptions.
Failed TestNoPendingDatabaseChanges [2 s]
Error Message:
System.Exception : CleanReturnAsync: Test killed the client in pair 4:

@NameLunar NameLunar closed this Jan 4, 2025
@NameLunar NameLunar reopened this Jan 4, 2025
@NameLunar NameLunar dismissed FaDeOkno’s stale review January 5, 2025 04:20

Сделал уже..

@NameLunar NameLunar requested a review from FaDeOkno January 5, 2025 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение локализации Changes: Map Изменение карт Changes: Sprites Изменение спрайтов S: Needs Review S: Untriaged size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants