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

[Prime Only] add event outfits #1531

Closed
wants to merge 15 commits into from
Closed

Conversation

Fullonibus
Copy link

@Fullonibus Fullonibus commented Sep 19, 2024

Что этот PR делает

Добавляет аутфиты для ивентовых персонажей прайма (прайм онли)

Почему это хорошо для игры

Легче настраивать ивентовых персонажей для раунда.

Изображения изменений

Тестирование

In my mind

Changelog

🆑
add: Добавлены аутфиты для ивентовых персонажей прайма
add: Добавлена косморуина для щитспавна: база СССП
fix: Была залита более темная версия плаща ЦК
/:cl:

@PhantornRU
Copy link
Collaborator

PhantornRU commented Sep 19, 2024

Линтер падает. Тестируй то что добавил сам.
Error OD0404 at modular_ss220/prime_only/code/outfit.dm:23:12: Type /obj/item/card/id/centcom/tanya does not exist

Copy link
Collaborator

@PhantornRU PhantornRU left a comment

Choose a reason for hiding this comment

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

Исправь карты и кучу других вещей:

modular_ss220/prime_only/code/outfit.dm:123:error: /obj/item/card/id/midnight: undefined type path
modular_ss220/prime_only/code/outfit.dm:23:error: /obj/item/card/id/centcom/tanya: undefined type path
modular_ss220/prime_only/code/outfit.dm:132:error: /obj/item/storage/belt/rapier/genri_rapier: undefined type path
modular_ss220/prime_only/code/outfit.dm:139:error: /obj/item/storage/belt/rapier/genri_rapier: undefined type path
modular_ss220/prime_only/code/outfit.dm:107:error: /obj/item/dualsaber/legendary_saber/normandy_saber: undefined type path
modular_ss220/prime_only/code/outfit.dm:23:12: Type /obj/item/card/id/centcom/tanya does not exist 

@github-actions github-actions bot added the 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. label Sep 19, 2024
Copy link
Collaborator

Choose a reason for hiding this comment

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

Руина только для ивента или должна быть в ротации? Если второе, то нужно обновить конфиг config.toml

Copy link
Author

Choose a reason for hiding this comment

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

Вообще в ротацию не планировалась. Изначально помещалась чтобы прогнать её по линтеру. В целом, может пригодится админам для ивентов с СССП базой (нам в том числе)

Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts left a comment

Choose a reason for hiding this comment

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

И вообще, может лучше тогда Outfit Manager очеловечить, чтобы не создавать датум для всех?

@github-actions github-actions bot added the 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! label Sep 20, 2024
Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts left a comment

Choose a reason for hiding this comment

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

Спрайту стало плохо, надо во всех стейтах цвет поменять. А вообще лучше это в отдельном ПРе сделать

@Fullonibus
Copy link
Author

а, это я пепега. Сейчас поправлю

@m-dzianishchyts m-dzianishchyts dismissed their stale review September 20, 2024 19:34

сделаны изменения

@github-actions github-actions bot added the Merge Conflict Ну блять... label Oct 2, 2024
Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts left a comment

Choose a reason for hiding this comment

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

Хорошо бы все обезличить (alt1, alt2, alt3, другие способы)

Copy link

This pull request seems to be stale as there have been no changes in 14 days, please make changes within 7 days or the PR will be closed. If you believe this is a mistake, please inform a development team member on Discord.

@github-actions github-actions bot added the Stale ПР долго был не активен, и требует обновления. label Oct 18, 2024
Copy link

This pull request has not received any updates since being marked stale, and as such is now being automatically closed. Please feel free to re-open this pull request or open a new one once you have new updates.

@github-actions github-actions bot closed this Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merge Conflict Ну блять... 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден Stale ПР долго был не активен, и требует обновления. 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants