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

InternalAffairs #1232

Closed
wants to merge 31 commits into from
Closed

Conversation

EstKemran
Copy link

@EstKemran EstKemran commented Jun 27, 2024

Описание PR

Я устал ждать КЗ и трудовой договор... Я хочу поиграть за департамент внутренних дел... Я хочу, что бы у нас была своя частота...

Медиа
изображение

изображение

изображение

изображение

изображение

изображение

изображение

Проверки

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

Изменения
🆑 Kemran

  • add: Добавлен департамент внутренних дел.
  • add: Добавлена частота и гарнитуры департамента внутренних дел.
  • tweak: Офицер "Синий Щит", Агент внутренних дел, Адвокат и Детектив перенесены в департамент внутренних дел.
  • tweak: Изменена форма в копировальном аппарате "Юридический отдел станциии" на "Департамент внутренних дел", с отличным от СБ цветом и бланком.

TODO: Возможно, ломает спавн адвоката
@github-actions github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" labels Jun 27, 2024
Лень делать локализацию + Не могу добавить на мапы
@github-actions github-actions bot added the Changes: Sprites Изменение затронуло файлы ".rsu / .png" label Jun 27, 2024
Copy link

github-actions bot commented Jun 27, 2024

RSI Diff Bot; head commit 94f30d8 merging into 67af4ad
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Interface/Misc/job_icons.rsi

State Old New Status
Detective Modified
Lawyer Modified

Resources/Textures/SS220/Clothing/Ears/Headsets/internal_affairs.rsi

State Old New Status
alt-equipped-EARS Added
equipped-EARS Added
icon Added
icon_alt Added

Resources/Textures/SS220/Objects/Devices/encryption_keys.rsi

State Old New Status
crypt_silver Added
lawyer_label Added

Edit: diff updated after 94f30d8

@UrPrice
Copy link
Collaborator

UrPrice commented Jun 27, 2024

@EstKemran Докрути идейку, предложи перенести Детектива в Департамент Внутренних Дел. Давно хотел такую штуку предложишь, обсуждали еще с Млаффи как-то очень давно в личке. А то в сервисе это конечно круто, но в Департаменте Внутренний Дел ему как-будто есть место.

Ну и PR у тебя сейчас встанет на месте, пока не пройдет этап согласований с ВикиЛороДрочью.

@EstKemran
Copy link
Author

@EstKemran Докрути идейку, предложи перенести Детектива в Департамент Внутренних Дел. Давно хотел такую штуку предложишь, обсуждали еще с Млаффи как-то очень давно в личке. А то в сервисе это конечно круто, но в Департаменте Внутренний Дел ему как-будто есть место.

Ну и PR у тебя сейчас встанет на месте, пока не пройдет этап согласований с ВикиЛороДрочью.

Он сейчас даже с адучем не согласован :xdd:

@EstKemran EstKemran changed the title InternalAffairs [Do not merge] InternalAffairs Jun 28, 2024
@FriedrichStrassen
Copy link

Кстати, в тринашке этот канал связи называется "Процессуальный", или в "Процедурный"

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Jul 23, 2024
Copy link

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

@github-actions github-actions bot added Merge Conflict Some content has conflicts. and removed Merge Conflict Some content has conflicts. labels Aug 4, 2024
Copy link

github-actions bot commented Aug 4, 2024

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

@github-actions github-actions bot removed the Merge Conflict Some content has conflicts. label Aug 5, 2024
@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Aug 6, 2024
@EstKemran EstKemran changed the title InternalAffairs [Do not merge] InternalAffairs Aug 6, 2024
@DexlerXD
Copy link

DexlerXD commented Aug 6, 2024

@EstKemran
image
ты локализацию не написал правильно, а назвал всё прямо в прототипах
оффы теперь запрещают это делать

@EstKemran
Copy link
Author

@EstKemran image ты локализацию не написал правильно, а назвал всё прямо в прототипах оффы теперь запрещают это делать

Ужас

@EstKemran
Copy link
Author

Чеки, вроде, по рофлу упали, так что все готово, наверное

@EstKemran EstKemran changed the title [Do not merge] InternalAffairs InternalAffairs Aug 8, 2024
@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Aug 9, 2024
Copy link

github-actions bot commented Aug 9, 2024

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

@github-actions github-actions bot removed the Merge Conflict Some content has conflicts. label Aug 9, 2024
@EstKemran
Copy link
Author

@Ady4ik

@gogenych
Copy link

А как чат улья попал в ДВД?

@Kirus59
Copy link
Collaborator

Kirus59 commented Aug 11, 2024

А как чат улья попал в ДВД?

Был замержен раньше

@Kirus59
Copy link
Collaborator

Kirus59 commented Aug 11, 2024

Он же отдельно, а не в двд
image

@@ -15,7 +15,7 @@
- Felinid #SS220-jobbans end
startingGear: IAAGear
icon: "JobIconIAA"
supervisors: job-supervisors-centcom
supervisors: job-supervisors-magistrate

Choose a reason for hiding this comment

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

добавить пометки

Choose a reason for hiding this comment

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

@@ -9,13 +9,14 @@
time: 72000 # SS220-RoleTime
startingGear: DetectiveGear
icon: "JobIconDetective"
supervisors: job-supervisors-hop
supervisors: job-supervisors-IAA

Choose a reason for hiding this comment

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

добавить пометки

Choose a reason for hiding this comment

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

@DexlerXD
Copy link

@EstKemran чини чеки, там скорее всего скрытый симмвол есть

@EstKemran EstKemran closed this Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: Sprites Изменение затронуло файлы ".rsu / .png"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants