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

NewRoles And Items #417

Closed
wants to merge 35 commits into from
Closed

NewRoles And Items #417

wants to merge 35 commits into from

Conversation

Rinary1
Copy link
Contributor

@Rinary1 Rinary1 commented Jan 12, 2024

Описание PR

Данный ПР включает в себя несколько новых ролей(в разработке, список может меняться):

  • "Синий Щит"
  • Магистрат

Случайно мёржнутый диван попал сюда, так что придеться добавить раньше срока

Медиа

Снимок экрана 2024-01-13 001931
Снимок экрана 2024-02-02 174930

Проверки

PR

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

Roles

  • Офицер Синий Щит
  • Магистрат

Изменения

🆑 Rinary

  • add: Добавлена Профессия Синего Щита
  • add: Добавлена Профессия Магистрата
  • add: Добавлен Диван

Rinary1 added 10 commits January 5, 2024 14:46
- Security - 50hrs
- Command - 50hrs
- Total - 250hrs
#Можно добавить больше, ибо чем больше часов нужно тем меньше челов которые просто будут дрочить в кабинетах глав и мешать антагам(ибо сами через это пройдут, или будут понимать что такое делать нельзя и за такое дают атата)
@nomad0260
Copy link
Collaborator

выглядит убедительно

@Rinary1
Copy link
Contributor Author

Rinary1 commented Jan 12, 2024

Синий щит фактически готов
(Не хватает только синего противогаза, для баланса вселенной)

@Rinary1
Copy link
Contributor Author

Rinary1 commented Feb 2, 2024

откатил до стабильной версии, буду делать магистрата, потом переведу из draft на проверку

Rinary1 and others added 4 commits February 2, 2024 14:49
- Adds Magistrat Job (Unstable)
- Added Magistrat Access
- Pocket filled
- Added to Departments
- Added Locale for Magistrat
TO DO:
- Spawn Markings
- All jobs Check
@Rinary1 Rinary1 marked this pull request as ready for review February 2, 2024 16:26
@Rinary1 Rinary1 requested a review from Rxup as a code owner February 2, 2024 16:26
@Rinary1 Rinary1 changed the title NewRoles NewRoles And Items Feb 2, 2024
@Rxup Rxup requested review from Kirillcas and removed request for Rxup February 11, 2024 12:52
@Kirillcas
Copy link
Contributor

Так, эти роли надо будет перекинуть в команду по типу CallSpecForces, ток по одиночно, и на гриду ЦК

@Rinary1
Copy link
Contributor Author

Rinary1 commented Feb 11, 2024

Так, эти роли надо будет перекинуть в команду по типу CallSpecForces, ток по одиночно, и на гриду ЦК

Зачем? В 13 они со старта на станции, разве нет?

Сомневаюсь что кто-то будет вызывать магистрата ради 1 суда, а синий щит вообще спецом отправляют на станцию для защиты глав, будет не логично добавлять их вызов

@Kirillcas
Copy link
Contributor

Зачем? В 13 они со старта на станции, разве нет?

и в каком билду 13 магистрат раундстартом? (впринципе наличие магистрата редкое в билду) ибо на моейм памяти так может выделиться ток парадайс, у нас жи по примеру и одного АВД хватает (и то, и к нему вопросы нужности), и делать больше ролей болванчиков как-то не охото.

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

Сомневаюсь что кто-то будет вызывать магистрата ради 1 суда, а синий щит вообще спецом отправляют на станцию для защиты глав, будет не логично добавлять их вызов

Аналогично, сомневаюсь что магистрат вообще нужен 24 на 7 на станции, если у него лишь одна смысловая нагрузка "суды"

а блющилд я уже сказал что идейно должен быть спец телохранителем сотрудников НТ, а не одевшимся в голубое секом и назначенный за определённое главой, но собственной ролью в лобби... кстати рили, никому не мешает так делать в раундах, просто дать возможность заказа голубой формы, хотя и так есть голубая рубашка и второй панцирь капитана... и ещё раз если так не терпеться голубых СБ под глав, делайте 3 повышения блющилда где ток первый будет под глав, вот когда им надо будет по больше безопасности, пусть заполняют документ на их прилёт.

@Rinary1
Copy link
Contributor Author

Rinary1 commented Feb 12, 2024

и в каком билду 13 магистрат раундстартом?

Я брал всё с Paradise Station.

Блющилда жи зачем-то кто-то идейно испоганил

Нет, его суть не испоганена, он и защищает сотрудников нт на станции, если быть конкретным то всё командование и вип персон.

Аналогично, сомневаюсь что магистрат вообще нужен 24 на 7 на станции, если у него лишь одна смысловая нагрузка "суды"

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

а блющилд я уже сказал что идейно должен быть спец телохранителем сотрудников НТ

Он им и есть, если говорить про цк то им вообще он не нужен ибо на цк есть офицеры цк которые сами могут убить кого хочешь

у нас жи по примеру и одного АВД хватает (и то, и к нему вопросы нужности), и делать больше ролей болванчиков как-то не охото.

А потом челы на сб смогут просто накидывать нарушения с пустых слов, и к кому потом обращаться? Ибо я уверен что капитан или гсб не будет идти в сб ради 1 записки на сб ( проверено )

@Rinary1 Rinary1 closed this Feb 22, 2024
@Rinary1
Copy link
Contributor Author

Rinary1 commented Feb 22, 2024

Закрываю ПР, Ибо нет смысла дальше ждать принятие, прошло уже 2 недели, так и не приняли решение, так-что видно что смысла делать что-то дальше нет. Тем более когда и так готовый пр пытаются переосмыслить в цк помойку.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants