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

Fixes for supplypacks #28

Closed
wants to merge 10 commits into from

Conversation

JohnChiffir
Copy link

About the Pull Request

Not much to say, cus everything is in changelog, so...keep yourself safe I guess <3

Why It's Good For The Game

Well adds sense for combat medic's role... funny that only EZ medics had something while other zones didn't have anything additional.
A supply has some great issues, I ain't a balancer or good coder and other and other, so I changed acceses for security stuff just at least make possible to open them.

Changelog

🆑
Adds some first aid kits to all combat medics
Adds separeted access for EZ and EZ Senior
Replaces accesses for security stuff in supply
Adds alt titles for Containment Engineer
/:cl:

@MysticalFaceLesS MysticalFaceLesS changed the base branch from dev to beta-dev April 10, 2024 19:05
Copy link

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

@MysticalFaceLesS MysticalFaceLesS changed the base branch from beta-dev to OldMainBranch April 13, 2024 07:36
@MrCat15352 MrCat15352 added 🚨Не мёржить!🚨 По каким-то причинам не мержить PR 🔎 Требует рассмотрения 🔍 🧱 Код 🧱 В PR есть код... labels Apr 28, 2024
@@ -85,6 +85,21 @@
mask = /obj/item/clothing/mask/balaclava
gloves = /obj/item/clothing/gloves/thick/swat/lcz

/decl/hierarchy/outfit/job/security/lcz_senmedic
Copy link
Collaborator

Choose a reason for hiding this comment

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

Прямо в кор код... У нас теперь работает система модульности с модпаками. С ней ты можешь ознакомиться прочитав README.md по пути mod_celadon/README.md

Copy link
Collaborator

Choose a reason for hiding this comment

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

@@ -17,7 +17,7 @@
cost = 30
containertype = /obj/structure/closet/crate/secure
containername = "light armor crate"
access = ACCESS_SECURITY
Copy link
Collaborator

Choose a reason for hiding this comment

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

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

@@ -144,6 +144,10 @@
economic_power = 5
minimal_player_age = 10
ideal_character_age = 30
alt_titles = list(
Copy link
Collaborator

Choose a reason for hiding this comment

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

Тут аналогично. Подскажу. Ты делаешь папку своего модпака, или вгоняешь в уже существующий модпак изменения. Чтобы сделать модпак есть скрипты, они валяются в mod_celadon запускаешь вводишь название и все. Потом уже тут пишешь

кор_код.....
// [CELADON-ADD] - ТВОЙ_ID_МОДПАКА - (тут комментарий можно и без)
на этих строках твой код добавленый...
...
...
// [/CELADON-ADD]

Будет вот так:

...
ideal_character_age = 30
// [CELADON-ADD] - CELADON_SUPPLY_PACKS
alt_titles = list(
	    "Containment Specialist",
		"Containment Technician"
		)
// [/CELADON-ADD]
outfit_type = /decl/hierarchy/outfit/job/engineering/conteng
....

@@ -222,7 +222,7 @@
supervisors = "the LCZ Zone Commander"
economic_power = 4
requirements = list("LCZ Guard" = 480)
alt_titles = list("LCZ Senior Containment Response Agent", "LCZ Containment Response Sergeant", "LCZ Senior Combat Medic", "LCZ Riot Control Sergeant", "LCZ Senior Riot Control Agent", "LCZ Senior Agent")
Copy link
Collaborator

Choose a reason for hiding this comment

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

А тут ты изменял. Тут уже надо будет использовать [CELADON-EDIT]

@@ -540,13 +540,20 @@ GLOBAL_LIST_EMPTY(conversion_cards)
item_state = "Sec_ID2"
job_access_type = /datum/job/ncoofficerlcz

/obj/item/card/id/seclvl3ez
/obj/item/card/id/seclvl3senez
Copy link
Collaborator

Choose a reason for hiding this comment

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

Изменение, но это объект, его можно перезаписать. Как работает. Ты оставляешь кор код. В модпаке тот же объект создаёшь и вписываешь только то что будет меняться у него. Или можешь тут изменить коркод

@MrCat15352 MrCat15352 added ⁉ Вопросы ⁉ Есть вопросы по PR 🛠 Доработать 🛠 PR нужно доработать. Есть комментарии или нет их... 📢Не модульно! Код не соответствует требованиям модульности labels May 4, 2024
@JohnChiffir JohnChiffir closed this May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⁉ Вопросы ⁉ Есть вопросы по PR 📢Не модульно! Код не соответствует требованиям модульности 🔎 Требует рассмотрения 🔍 🚨Не мёржить!🚨 По каким-то причинам не мержить PR 🛠 Доработать 🛠 PR нужно доработать. Есть комментарии или нет их... 🧱 Код 🧱 В PR есть код...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants