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

Роль - Магистрат #1711

Closed
wants to merge 15 commits into from
Closed

Conversation

Agoichi
Copy link
Contributor

@Agoichi Agoichi commented Jan 2, 2024

Описание PR

Начну с того почему я сделал Pull Request Корваксу а не Волшебникам
Это дело принципа.
Волшебники взяли курс и держат его прямиком в пучины лрп. Соответствено любая предложка которая будет нацеленна на улучшение мрп будет откланятся за ненадобностью для них.
Наш проект корвакс позиционирует себя как МРП и следовать во всем лрп проекту я считаю крайне не дальновидным.

Мой ПР добавляет новую роль в пул ролей, элементы одежды, кпк, гарнитуру, ID карту, молот судьи и новый штамп.
Также вносит изменения в прототип карт для внедрения новой роли
Роль - Магистрат.

Почему эта наработка может быть внедрена в корвакс и не нарушает логику и атмосферу оного:
Так как мне много чего есть сказать я приведу ссылку на предложение в канале сервера корвакс которое я создал в поддержку этому пр-у, и в котором приведен почти каждый аргумент который у меня имеется.
Ссылка на предложку на Дискорд сервере корвакс: ⁠https://discord.com/channels/919301044784226385/1190065917518938122

.. если хотите можете даже сделать спонсорской ролью.. хехе... (ненадо)

Медиа
image

Проверки

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

Изменения

  • add: Добавлена новая роль - Магистрат
  • add: Добавлены элементы одежды магистрата (Парик магистрата, мантия магистрата)
  • add: Добавлены девайсы (Гарнитура магистрата, КПК магистрата с ручкой центком + ID карта магистрата )
  • add: Добавлен судейский молот магистрата (добавлены также звуки удара)
  • add: Добавлен штамп магистрата
  • tweak: Изменены прототипы карт (Добавлин магистрат в них. В карты корвакса с возможностью раундстарта, для карт оффов только с поздним подключением из-за политики не маппинга оффовских карт)

🆑

  • add: Добавлен магистрат и его одежда!
  • add: Добавлен молот магистрата со встроенным предохранителем!
  • add: Добавлен штамп магистрата!

- парик, мантия, кпк, гарнитура, айдикарта, молот
- Роль
- Звук
- Перевод
@github-actions github-actions bot added Status: Needs Review This PR requires new reviews before it can be merged Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation Changes: Map Can be reviewed or fixed by people who are knowledgeable with mapping Changes: Sprites Should be reviewed or fixed by people who are knowledgeable with spriting or visual design Changes: No C# Code has not been changed labels Jan 2, 2024
Copy link
Contributor

github-actions bot commented Jan 2, 2024

RSI Diff Bot; head commit 259e0e9 merging into 7be2e66
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Corvax/Clothing/Ears/Headsets/judge.rsi

State Old New Status
equipped-EARS Added
icon Added

Resources/Textures/Corvax/Clothing/Head/Hats/magr_hair.rsi

State Old New Status
equipped-HELMET Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Corvax/Clothing/OuterClothing/misc/magistrat.rsi

State Old New Status
equipped-OUTERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Corvax/Objects/Misc/stamps.rsi

State Old New Status
stamp-judge Added

Resources/Textures/Interface/Misc/job_icons.rsi

State Old New Status
Judge Added

Resources/Textures/Markers/jobs.rsi

State Old New Status
judge Added

Resources/Textures/Objects/Devices/pda.rsi

State Old New Status
pda-judge Added

Resources/Textures/Objects/Misc/id_cards.rsi

State Old New Status
idjudge Added

Resources/Textures/Objects/Weapons/Melee/judge_hammer.rsi

State Old New Status
icon Added
inhand-left Added
inhand-right Added

Edit: diff updated after 259e0e9

Copy link
Collaborator

@lzk228 lzk228 left a comment

Choose a reason for hiding this comment

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

  1. Меня волнует то, что в прототипах и локали разные описания.
  2. В описаниях ссылается на другие професии, как-то не очень в целом выглядит.
  3. Магистрат, везде в файлах судья почему-то.
  4. Пожалуйста, никаких раундстартовых магистратов на картах корвакса. Люди будут выбирать карты потому что они лучше и будет неравновесие в картах.

@778b
Copy link
Contributor

778b commented Jan 2, 2024

Ну скорее надо было делать ПР волшебникам, у корваксов все таки парадигма что они АДАПТИРУЮТ контент волшебников для РУ комьюнити, поэтому твой ПР скорее всего побреют

@Nimfar11
Copy link
Contributor

Nimfar11 commented Jan 2, 2024

Я "за" хотябы ради эксперимента, как оно покажет себя в деле, а не виртуальном размышлении. Но проблема размещения Магистрата останется, ведь "своей" комнаты у него не будет. Разве что сидеть в комнате АВД.
И разве Магистрату нужен "Внешний" доступ?

@lapatison
Copy link
Contributor

Почему магистрат, а не судья?

@Agoichi
Copy link
Contributor Author

Agoichi commented Jan 3, 2024

Почему магистрат, а не судья?

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

@github-actions github-actions bot added the Merge Conflict PR have unsolved merge conflicts label Jan 10, 2024
Copy link
Contributor

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

Agoichi and others added 2 commits January 10, 2024 12:31
убрал барратри
обновил трекер ролей
@github-actions github-actions bot removed the Merge Conflict PR have unsolved merge conflicts label Jan 10, 2024
Copy link
Contributor

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

@github-actions github-actions bot added the Merge Conflict PR have unsolved merge conflicts label Jan 30, 2024
@MureixloI
Copy link
Contributor

Почему магистрат, а не судья?

Звучит брутальнее, а также на вики написано как магистрат

@Kit0vras
Copy link
Contributor

Главное не добавлять его в раундстарт-роли, ибо магистрат это дядечка, который 90% смены сидит и пинает балду, потому что ему попросту нечего делать. Только для ивентов

@Deserty0
Copy link
Contributor

Главное не добавлять его в раундстарт-роли, ибо магистрат это дядечка, который 90% смены сидит и пинает балду, потому что ему попросту нечего делать. Только для ивентов

Так прототипом всегда можно, смысл пра добавление дядечки Магистрата в структуру процедурного ака юридического отдела

@Kit0vras
Copy link
Contributor

Главное не добавлять его в раундстарт-роли, ибо магистрат это дядечка, который 90% смены сидит и пинает балду, потому что ему попросту нечего делать. Только для ивентов

Так прототипом всегда можно, смысл пра добавление дядечки Магистрата в структуру процедурного ака юридического отдела

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

@MureixloI
Copy link
Contributor

Главное не добавлять его в раундстарт-роли, ибо магистрат это дядечка, который 90% смены сидит и пинает балду, потому что ему попросту нечего делать. Только для ивентов

Так прототипом всегда можно, смысл пра добавление дядечки Магистрата в структуру процедурного ака юридического отдела

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

@Deserty0
Copy link
Contributor

Главное не добавлять его в раундстарт-роли, ибо магистрат это дядечка, который 90% смены сидит и пинает балду, потому что ему попросту нечего делать. Только для ивентов

Так прототипом всегда можно, смысл пра добавление дядечки Магистрата в структуру процедурного ака юридического отдела

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

по решению суда присяжных, вы приговорены к каторге на клоунской станции хонк

Теперя мапперы сами решат есть ли магистрат раундстартом у них
@github-actions github-actions bot removed the Merge Conflict PR have unsolved merge conflicts label Jan 31, 2024
@Morb0 Morb0 closed this Feb 7, 2024
@Dmitry2777
Copy link

Круто

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation Changes: Map Can be reviewed or fixed by people who are knowledgeable with mapping Changes: No C# Code has not been changed Changes: Sprites Should be reviewed or fixed by people who are knowledgeable with spriting or visual design Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants