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

Фиксы #211

Merged
merged 30 commits into from
Jul 18, 2024
Merged

Фиксы #211

merged 30 commits into from
Jul 18, 2024

Conversation

Rinary1
Copy link
Contributor

@Rinary1 Rinary1 commented Jul 17, 2024

Кратное описание

  • Добавляет санрайзовским эмоциям вайтлисты а так-же переводит некоторые.

  • Так-же есть 2 новых паралакса

  • Удаляеться attributions.yml в музыке Санрайза, так как музыку удалили, а файл с пустым листом вызывал ошибку

  • Убераються все юзы удалённой музыки

  • Фикс консоли криминальных записей(нет ItemToggleComponent, зачем-то юзает 2 парента, когда есть Base от оффов)

  • Фикс cluwne-name-prefix, пытался использовать переменную target которой не существует для него.

  • Фикс Combat Mode Action, когда тест добавлял экшены, он тестил их пытаясь получить True параметр, а так как дефолт значение комбат мода false он ловил ошибку, изменим тест, что-бы работал через try

  • Фикс робоанализатора, аналогично криминальным записям, только удалён вариант Unpowered

  • обновил список проверок карт

  • на оффовских картах нет спавнера магистрата, юриста, юридический отдел закомментирован

  • добавлена новая проверка при спавне ЦК, раньше возникала ошибка что Centcomm не может быть null, теперь если ЦК null, оно так-же не заспавниться и будет выводить ошибку в 1 строчку а не 100

  • добавлена гарнитура юриста

  • у авд убран канал коммандования

По какой причине

Без вайтлистов любой моб, даже тот который не мог говорить или издавать этот звук, мог использовать эти эмоции

Медиа(Видео/Скриншоты)

Проверки

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

Changelog

🆑 Rinary

  • fix: Исправлены эмоции, более они не будут работать у расс/мобов для которых они не предназначенны.
  • tweak: Эмоции вульп, троттинов были переведены.
  • add: Добавлена гарнитура юриста.
  • remove: Удалена частота коммандования у АВД (зачем?).

@Rinary1 Rinary1 changed the title Фикс Эмоций Фиксы Jul 17, 2024
@Rinary1 Rinary1 changed the title Фиксы Фиксы, чистка остатков музыки Jul 17, 2024
@SplikZerys
Copy link
Collaborator

Я по фасту музыку удалил, чтоб сегодня обновить все сервера, пофикси конфликты потом пожалуйста

@SplikZerys
Copy link
Collaborator

Потом Вигерс ревью сделает

Copy link
Contributor

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

…o FixEmotes

# Conflicts:
#	Resources/Audio/_Sunrise/Music/attributions.yml
@Rinary1 Rinary1 changed the title Фиксы, чистка остатков музыки Фиксы Jul 17, 2024
@Rinary1
Copy link
Contributor Author

Rinary1 commented Jul 17, 2024

мейн ошибки я победил, другие пофикшу утром или ещё позже

@Rinary1
Copy link
Contributor Author

Rinary1 commented Jul 18, 2024

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

@Rinary1
Copy link
Contributor Author

Rinary1 commented Jul 18, 2024

можно ревьювить ибо как фиксить остачу я не знаю, только залазить в тесты

Copy link
Collaborator

@VigersRay VigersRay left a comment

Choose a reason for hiding this comment

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

Раз уж ты залез в прототип робоанализатора, вместо удаления закомиченой части сделай так чтобы его видуал работать аналогично анализатору здоровья. (После чего я закину его оффам)

@Rinary1
Copy link
Contributor Author

Rinary1 commented Jul 18, 2024

Раз уж ты залез в прототип робоанализатора, вместо удаления закомиченой части сделай так чтобы его видуал работать аналогично анализатору здоровья. (После чего я закину его оффам)

окей

Copy link
Contributor

github-actions bot commented Jul 18, 2024

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

Resources/Textures/_Sunrise/roboanalyzer.rsi

State Old New Status
icon Modified
analyzer Added

Edit: diff updated after d72c885

@VigersRay VigersRay merged commit b562c25 into space-sunrise:master Jul 18, 2024
8 of 13 checks passed
@Rinary1 Rinary1 deleted the FixEmotes branch July 19, 2024 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants