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.

Sign up for GitHub

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

Jump to bottom

[ПЕРЕНОС] Ведущие роли #374

Merged
merged 19 commits into from
Aug 31, 2024
Merged

[ПЕРЕНОС] Ведущие роли #374

merged 19 commits into from
Aug 31, 2024

Conversation

Darkiich
Copy link
Contributor

@Darkiich Darkiich commented Aug 30, 2024 •
edited
Loading

Описание PR

Ссылка на публикацию в Discord

Медиа

Проверки

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

Роли:

  • Ведущий врач
  • Доцент
  • Бригадир
  • Инструктор СБ
    image
    image
    image

Изменения

🆑

  • add: Добавлены ведущие роли - Доцент, Ведущий врач, Бригадир, Офицер-Инструктор

@github-actions GitHub Actions
Copy link
Contributor

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

@github-actions github-actions bot added No C# Changes: Localization Изменение локализации labels Aug 30, 2024
@Schrodinger71
Copy link
Collaborator

image
Оставляй ссылку в скобка ( )
тогда текст в [ ] превратиться в гипперссылку

@Darkiich
Copy link
Contributor Author

image Оставляй ссылку в скобка ( ) тогда текст в [ ] превратиться в гипперссылку

okkkkkkk

@COTE-LAPYX
Copy link

Мне кажется или это дубликат кода?
Resources/Locale/ru-RU/ADT/Job/job-names.ftl
Screenshot_2024-08-30-14-15-24-140-edit_com.github.android.jpg

@COTE-LAPYX
Copy link

31 строка
Resources/Prototypes/ADT/StatusIcon/job.yml
Возможно стоит указать jobName

@Darkiich
Copy link
Contributor Author

Че я за хуйню сделал

@Bolper Bolper added the In progress В процессе выполнения label Aug 30, 2024
@Darkiich Darkiich self-assigned this Aug 31, 2024
@Darkiich Darkiich linked an issue Aug 31, 2024 that may be closed by this pull request
Роли "сеньоров" #265
Closed
@Darkiich Darkiich requested a review from 1Stepka1 August 31, 2024 10:00
@Darkiich Darkiich requested review from PyotrIgn and Bolper August 31, 2024 10:00
Bolper
Bolper reviewed Aug 31, 2024
View reviewed changes
Copy link
Contributor

@Bolper Bolper left a comment

Choose a reason for hiding this comment

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

Если честно всё ок. Но апрувать не смогу потому что вдруг что недоглядел с телефона. Придётся ждать вечера. Но очень хорошо все

Resources/Prototypes/ADT/Roles/Jobs/Engineering/senior_engineer.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Roles/Jobs/departments.yml Show resolved Hide resolved
@Bolper Bolper self-requested a review August 31, 2024 11:15
1Stepka1
1Stepka1 reviewed Aug 31, 2024
View reviewed changes
Copy link
Collaborator

@1Stepka1 1Stepka1 left a comment

Choose a reason for hiding this comment

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

И добавить все роли на карты а то тесты валяться что их нету на картах и плодят,или это потом сделаете?

Resources/Prototypes/ADT/Roles/Jobs/Security/senior_officer.yml Show resolved Hide resolved
@Bolper
Copy link
Contributor

Bolper commented Aug 31, 2024

И добавить все роли на карты а то тесты валяться что их нету на картах и плодят,или это потом сделаете?

Да, от того что на картах нет спавнов данных ролей, то тесты уходят в небытие и жалуются на это

@Darkiich
Copy link
Contributor Author

И добавить все роли на карты а то тесты валяться что их нету на картах и плодят,или это потом сделаете?

Уже добавил, осталось спавнеры

@Bolper
Copy link
Contributor

Bolper commented Aug 31, 2024

И добавить все роли на карты а то тесты валяться что их нету на картах и плодят,или это потом сделаете?

Уже добавил, осталось спавнеры

Лучше убери. Тебе ещё придётся сравнены расставлять тогда. Иначе не пустит

@Darkiich
Copy link
Contributor Author

И добавить все роли на карты а то тесты валяться что их нету на картах и плодят,или это потом сделаете?

Уже добавил, осталось спавнеры

Лучше убери. Тебе ещё придётся сравнены расставлять тогда. Иначе не пустит

капессс

Bolper
Bolper reviewed Aug 31, 2024
View reviewed changes
Copy link
Contributor

@Bolper Bolper left a comment

Choose a reason for hiding this comment

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

Я ничего не посмотрел, патаму чта фиксь ямл

Resources/Prototypes/ADT/Entities/Markers/Spawners/jobs.yml Outdated Show resolved Hide resolved
1Stepka1
1Stepka1 approved these changes Aug 31, 2024
View reviewed changes
Copy link
Collaborator

@1Stepka1 1Stepka1 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Ну вроде в порядке все, только тесты остались

@Darkiich Darkiich added this pull request to the merge queue Aug 31, 2024
Merged via the queue into AdventureTimeSS14:master with commit 6a021eb Aug 31, 2024
11 checks passed
@Darkiich Darkiich deleted the senior branch August 31, 2024 15:39
@jungarikjan jungarikjan mentioned this pull request Dec 10, 2024
Merged
PyotrIgn added a commit that referenced this pull request Dec 10, 2024
@PyotrIgn
[FIX] Одежда ведущих должностей в лоудаутах + мелкие фиксы (#883)
5fd6088 
## Описание PR
Береты врачам, инженерам, ученым достаются за наигранное время
У врачей нет хирургической одежды в лоудауте. Пусть берут только для
операций и в специальном для этого автомате.
У врачей, инженеров и учёных пропала одежда ведущих должностей (доцента,
ведущего врача, бригадира) и робототехника (у учёных) из лоудаутов
Фикс и добавление локали
Фикс спавнера Поппи (спавнился Морти)
Халаты врачей убраны у учёных (у них свои есть)
Тех. ассистент переименован в инженера-стажёра (окончательно)
Научный ассистент переименован в лаборанта (окончательно)

## Почему / Баланс
Потому что люди абузили одежду ведущих должностей, а обязанности их не
выполняли.
Фиксит следующие ПРы:
#160
#374

## Техническая информация
Ну... Лоудауты - это пиздец. Но я прибрался там.
Комментарии не везде поставил, и не всё комментировал. То, что быть не
должно было в файле - удалил и перенёс в другой файл на его законное
место. В локали комменты не оставлял, ибо они потом отображаются в
описании и т.д. и т.п.

## Медиа
Вот это я переименовал в "доцент" и "бригадир" соответственно

![image](https://github.com/user-attachments/assets/95c3812c-8d54-4553-b1b3-fb28722ae655)
"Научрука" заменил на "научного руководителя" и т.д.

![image](https://github.com/user-attachments/assets/f9c84557-8f74-4b2a-8d3b-8502302318a5)
БЫЛО:


![image](https://github.com/user-attachments/assets/aa8e88f6-fc15-4205-9aeb-2c15190bead9)

СТАЛО:


![image](https://github.com/user-attachments/assets/6805f979-b8ca-427a-b523-a13f0491f9b0)

И так у учёного ещё и у инженера. У всех должностей соответствующая им
униформа.



## Требования
- [Х] Я прочитал(а) и следую [Руководство по созданию пулл
реквестов](https://docs.spacestation14.com/en/general-development/codebase-info/pull-request-guidelines.html).
Я понимаю, что в противном случае мой ПР может быть закрыт по усмотрению
мейнтейнера.
- [Х] Я добавил скриншоты/видео к этому пулл реквесту, демонстрирующие
его изменения в игре, **или** этот пулл реквест не требует демонстрации
в игре

## Критические изменения
Отсутствуют

**Чейнджлог**
:cl: eddiemercury
- tweak: Береты врачам, инженерам, ученым достаются за наигранное время.
У ведущих и глав (у кого нет своих беретов) - по дефолту
- tweak: У врачей нет хирургической одежды в лоудауте. Пусть берут
только для операций и в специальном для этого автомате.
- fix: У врачей, инженеров и учёных пропала одежда ведущих должностей
(доцента, ведущего врача, бригадира) и робототехника (у учёных) из
лоудаутов
- fix: Фикс и добавление локали
- fix: Фикс спавнера Поппи (спавнился Морти)
- tweak: Халаты врачей убраны у учёных (у них свои есть)
- fix: Тех. ассистент переименован в инженера-стажёра (окончательно)
- fix: Научный ассистент переименован в лаборанта (окончательно)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bolper Bolper Bolper left review comments

@Filokini Filokini Filokini left review comments

@1Stepka1 1Stepka1 1Stepka1 approved these changes

@PyotrIgn PyotrIgn Awaiting requested review from PyotrIgn

Assignees

@Darkiich Darkiich

Labels
Changes: Localization Изменение локализации In progress В процессе выполнения No C# Status: Needs Review
Projects
None yet
Milestone
Должности станции
Development

Successfully merging this pull request may close these issues.

Роли "сеньоров"
6 participants
@Darkiich @Schrodinger71 @COTE-LAPYX @Bolper @Filokini @1Stepka1