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

Магистрат и юридический департамент #44

Merged
merged 16 commits into from
Jun 8, 2024

Conversation

Rinary1
Copy link
Contributor

@Rinary1 Rinary1 commented Jun 7, 2024

About the PR

Данный пр добавляет роль магистрата.

Why / Balance

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

Media

image
image

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

  • add: Была добавлена роль магистрата

Rinary1

This comment was marked as outdated.

Copy link
Collaborator

@VigersRay VigersRay left a comment

Choose a reason for hiding this comment

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

Добавь юридический департамент, закинь туда авд, магистрата и юриста.
Включи для всех станций юриста и сделай его доступным для выбора.

@VigersRay
Copy link
Collaborator

Ещё только что заметил что магистрат не добавлен как роль на карты, это в прототипах станций.

@Rinary1 Rinary1 requested a review from VigersRay June 8, 2024 19:32
@Rinary1
Copy link
Contributor Author

Rinary1 commented Jun 8, 2024

По идеи сделал всё как просил, если что-то упустил, анлак, доделаю позже

@Rinary1 Rinary1 changed the title Magistrat Job Magistrat Job + Law Department Jun 8, 2024
@VigersRay
Copy link
Collaborator

По идеи сделал всё как просил, если что-то упустил, анлак, доделаю позже

image
Одного юриста мало, обыно их 2-3 (зависит от размера станции), и удали закомиченых юристов раз ты их вынес друг к другу.

@VigersRay VigersRay changed the title Magistrat Job + Law Department Магистрат и юридический департамент Jun 8, 2024
@Rinary1
Copy link
Contributor Author

Rinary1 commented Jun 8, 2024

По идеи сделал всё как просил, если что-то упустил, анлак, доделаю позже

image Одного юриста мало, обыно их 2-3 (зависит от размера станции), и удали закомиченых юристов раз ты их вынес друг к другу.

Готово

@VigersRay
Copy link
Collaborator

По идеи сделал всё как просил, если что-то упустил, анлак, доделаю позже

image Одного юриста мало, обыно их 2-3 (зависит от размера станции), и удали закомиченых юристов раз ты их вынес друг к другу.

Готово

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

@VigersRay
Copy link
Collaborator

И конфликты реши...

@Rinary1
Copy link
Contributor Author

Rinary1 commented Jun 8, 2024

И конфликты реши...

решил

… Roles

# Conflicts:
#	Resources/Prototypes/_Sunrise/Maps/box.yml
#	Resources/Prototypes/_Sunrise/Maps/cluster.yml
#	Resources/Prototypes/_Sunrise/Maps/fland.yml
#	Resources/Prototypes/_Sunrise/Maps/marathon.yml
@VigersRay
Copy link
Collaborator

И конфликты реши...

решил

А на наши (_Sunrise/Maps/) карты роли добавить?

@Rinary1
Copy link
Contributor Author

Rinary1 commented Jun 8, 2024

Секунду, ещё решаю конфликты

@Rinary1
Copy link
Contributor Author

Rinary1 commented Jun 8, 2024

готово

@VigersRay VigersRay merged commit e7b1402 into space-sunrise:master Jun 8, 2024
5 of 11 checks passed
@Rinary1 Rinary1 deleted the Roles branch June 8, 2024 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants