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

[FEAT] Centcom bat and reclinable revolvervs port #449

Closed
wants to merge 23 commits into from

Conversation

Chorden523
Copy link

@Chorden523 Chorden523 commented Jul 22, 2024

About The Pull Request

Портирует биту ЦК, РШ-12 и Анаконду с пары.

Why It's Good For The Game

Админабуз это весело

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

image
image
image
image
image

Changelog

🆑
add: Добавлен РШ-12 и Анаконда
add: Добавлена бита ЦК
/:cl:

@github-actions github-actions bot added 🔉 Звук Нам скорее всего нравится как это звучит. 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! labels Jul 22, 2024
@Furrior
Copy link
Collaborator

Furrior commented Jul 22, 2024

Мб биту сделать двуручной? Тогда не придется плодить доп переменные


/obj/item/melee/baseball_bat/homerun/central_command/pickup(mob/living/carbon/human/user)
. = ..()
if(!istype(user.get_item_by_slot(ITEM_SLOT_ID), /obj/item/card/id/advanced/centcom))
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
Author

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.

Через роль в майнде мб? Я помню частые случаи, когда антаги пиздили карты ерт, потом оцк пизды давали 🐁

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
Collaborator

Choose a reason for hiding this comment

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

Как варик, тупо проверять на админку

Copy link
Author

Choose a reason for hiding this comment

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

Ну, в майнде как на паре нет offstation_role и у самих ЦКшников нет как такового отличия от простых смертных кроме айдишки. Ну в общем посмотрю ещё может что найду.

Copy link
Author

Choose a reason for hiding this comment

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

Как варик, тупо проверять на админку

Изначально кстати так и думал, но потом подумал про то что на ПОЦК могут и простых игроков сажать.

@Chorden523
Copy link
Author

Мб биту сделать двуручной? Тогда не придется плодить доп переменные

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

@Chorden523 Chorden523 closed this Jul 25, 2024
@Chorden523 Chorden523 reopened this Jul 30, 2024
@@ -46,7 +46,7 @@

/obj/item/melee/baseball_bat/homerun/central_command/pickup(mob/living/carbon/human/user)
. = ..()
if(!istype(user.get_item_by_slot(ITEM_SLOT_ID), /obj/item/card/id/advanced/centcom))
if(!istype(user.get_item_by_slot(ITEM_SLOT_ID), /obj/item/card/id/advanced/centcom) || subtypesof(/obj/item/card/id/advanced/centcom))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ты же в курсе, что subtypesof(/obj/item/card/id/advanced/centcom всегда будет TRUE?

@Chorden523 Chorden523 marked this pull request as draft August 9, 2024 13:11
Copy link

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale label Aug 18, 2024
@Gaxeer Gaxeer removed the Stale label Aug 22, 2024
Copy link

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale label Aug 30, 2024
@github-actions github-actions bot closed this Sep 7, 2024
@Chorden523 Chorden523 deleted the admin_weapons_port branch December 24, 2024 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 🔉 Звук Нам скорее всего нравится как это звучит.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants