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

Add: Quartermaster as head of the department #4817

Merged
merged 28 commits into from
Apr 11, 2024
Merged

Add: Quartermaster as head of the department #4817

merged 28 commits into from
Apr 11, 2024

Conversation

NotSoWizard
Copy link
Contributor

@NotSoWizard NotSoWizard commented Apr 6, 2024

Описание

Так или иначе, мы подходим к тому моменту, что все наши конкуренты давно ввели Квартирмейстера как полноценного Главу отделения, чем мы хуже? Последняя клавиша написанного кода поставила жирную точку в этой истории. Я снял палец с клавиатуры - всё было кончено.

  • Теперь Квартирмейстер будет являться полноценным Главой отделения, имеющим доступ на Мостик, Бриг (базовый доступ), хранилище боеголовки, EVA, персональные шкафчики, и RC анонсы.
  • Квартирмейстер теперь может уволить коллегу по цеху, не недомогая Главу Персонала.
  • Квартирмейстер теперь может надоедать остальным Главам в радиоканал
  • Глава Персонала теперь не имеет доступа в Карго, но до сих пор может брать ресурсы.
  • Глава Персонала теперь не может использовать Civilian Door Remote для управления дверьми Карго.
  • Квартирмейстер теперь не может быть революционером, и сам является мишенью.

Спрайт для экрана не особо новой консоли был одобрен Ведущим Спрайтером FanTik.
Изменения карты были одобрены новым Бригадиром Мапперов a_e_t_e_r_n_a (AKA "SQUEEK!").
Спасибо им обоим, и Vladisvell, что помог в починкой кода.

Ссылка на предложение/Причина создания ПР

https://discord.com/channels/617003227182792704/755125334097133628/1215921517217579078

Демонстрация изменений

image
image
image
image
image
image

@github-actions github-actions bot added MAP Affects map DMI Affects DMI files labels Apr 6, 2024
@NotSoWizard NotSoWizard changed the title Add: Quartermaster as an Head of department Add: Quartermaster as head of the department Apr 6, 2024
@NotSoWizard NotSoWizard marked this pull request as ready for review April 6, 2024 23:56
@NotSoWizard NotSoWizard requested a review from a team as a code owner April 6, 2024 23:56
@ArtemDerp
Copy link
Contributor

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

@NotSoWizard
Copy link
Contributor Author

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

  1. Изменения на Дельте согласованы с бригадиром aeterna0_ (SQUEEK!). Оригинальный автор был против создания ноутбука как замену консоли.

image

Copy link
Contributor

@Rerik007 Rerik007 left a comment

Choose a reason for hiding this comment

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

const HeadRoles = [
"Captain",
"Head of Security",
"Chief Engineer",
"Chief Medical Officer",
"Research Director",
"Head of Personnel",
];
// Head colour check. Abbreviated to save on 80 char
const HCC = role => {
// Return green if they are the head
if (HeadRoles.indexOf(role) !== -1) {
return "green";
}
// Return yellow if its the qm
if (role === "Quartermaster") {
return "yellow";
}
// Return orange if its a regular person
return "orange";
};
// Head bold check. Abbreviated to save on 80 char
const HBC = role => {
// Return true if they are a head, or a QM
if ((HeadRoles.indexOf(role) !== -1) || role === "Quartermaster") {

Тут требуется добавить в лист глав и убрать отображение кма жёлтым.

Copy link
Contributor

@Rerik007 Rerik007 left a comment

Choose a reason for hiding this comment

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

Пара моментов.

code/game/gamemodes/revolution/revolution.dm Outdated Show resolved Hide resolved
code/game/jobs/jobs.dm Show resolved Hide resolved
@github-actions github-actions bot added TGUI Affects TGUI javascript Pull requests that update Javascript code labels Apr 8, 2024
@github-actions github-actions bot added the Merge Conflict This doesn't seem right label Apr 9, 2024
Rerik007
Rerik007 previously approved these changes Apr 9, 2024
Copy link
Contributor

@Rerik007 Rerik007 left a comment

Choose a reason for hiding this comment

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

Синий фон глав стандартен, взять бы его тоже. А так всё окей как по мне. Нужно только убрать изменения из комнат отеля, они сделаны в #4836

@github-actions github-actions bot removed the Merge Conflict This doesn't seem right label Apr 9, 2024
@NotSoWizard NotSoWizard requested a review from Rerik007 April 9, 2024 09:17
Rerik007
Rerik007 previously approved these changes Apr 9, 2024
@github-actions github-actions bot added the Merge Conflict This doesn't seem right label Apr 9, 2024
@NotSoWizard NotSoWizard closed this Apr 9, 2024
@NotSoWizard NotSoWizard reopened this Apr 9, 2024
@github-actions github-actions bot removed the Merge Conflict This doesn't seem right label Apr 9, 2024
@NotSoWizard NotSoWizard requested a review from Rerik007 April 9, 2024 14:42
Rerik007
Rerik007 previously approved these changes Apr 9, 2024
@Gottfrei
Copy link
Contributor

в задачах на воровство нужно убрать ГП из целек в отделе КМа.

@pilygun
Copy link
Contributor

pilygun commented Apr 11, 2024

Нужно перенести ваучер на получения шахтерской санряги с мостика, в шкафчик КМа. А так же нужно добавить в его шкаф капсулу, ибо удаляя из шкафов капсулы шахтеров обещали добавить 1 КМу но не добавили.

@NotSoWizard
Copy link
Contributor Author

NotSoWizard commented Apr 11, 2024

в задачах на воровство нужно убрать ГП из целек в отделе КМа.

Изменил.

@NotSoWizard
Copy link
Contributor Author

Рано или поздно этот ПР будет замерджен. Надеюсь.

aeternaclose
aeternaclose previously approved these changes Apr 11, 2024
@NotSoWizard NotSoWizard requested a review from Rerik007 April 11, 2024 16:05
@Rerik007 Rerik007 added this pull request to the merge queue Apr 11, 2024
Merged via the queue into ss220-space:master220 with commit 88501cf Apr 11, 2024
10 checks passed
@NotSoWizard NotSoWizard deleted the QM-promotion2 branch April 11, 2024 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DMI Affects DMI files javascript Pull requests that update Javascript code MAP Affects map TGUI Affects TGUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants