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

SeniorService rename #2093

Merged
merged 1 commit into from
Oct 15, 2024
Merged

SeniorService rename #2093

merged 1 commit into from
Oct 15, 2024

Conversation

Tris0r
Copy link

@Tris0r Tris0r commented Oct 15, 2024

Описание PR

Переименование ведущего сервисного работника в сервисного администратора

Медиа

Проверки

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

Изменения
🆑 Cortez

  • add: Ведущий сервисного отдела переименован в сервисного администратора

@Tris0r
Copy link
Author

Tris0r commented Oct 15, 2024

Может что то и забыл, сильно не бейте...

@github-actions github-actions bot added the Changes: Localization Изменение затронуло файлы ".ftl" label Oct 15, 2024
Kirus59
Kirus59 reviewed Oct 15, 2024
View reviewed changes
Resources/Locale/ru-RU/ss220/job/job-description.ftl
@@ -1,7 +1,7 @@
job-description-senior-officer = Инструктор СБ должен обучать кадетов или офицеров основным профессиональным навыкам отдела.
job-description-senior-officer = Инструктор СБ должен обучать кадетов или офицеров основным профессиональным навыкам отдела.
Copy link
Collaborator

Choose a reason for hiding this comment

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

?

Copy link
Author

Choose a reason for hiding this comment

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

Я хрен его знает почему эти строки как-то затронулись.

Resources/Prototypes/SS220/Entities/Clothing/Uniforms/Jumpskirt/jumpskirts.yml
sprite: SS220/Clothing/Uniforms/Jumpskirt/whitecaptain.rsi
Copy link
Collaborator

Choose a reason for hiding this comment

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

?

Copy link
Author

Choose a reason for hiding this comment

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

Я хрен его знает почему этот файл как-то затронулся.

@Kirus59 Kirus59 self-assigned this Oct 15, 2024
@Kirus59 Kirus59 requested a review from Ady4ik October 15, 2024 05:29
@UrPrice UrPrice added the Need to add Wiki220 Требуется добавить на Wiki/поставить задачу Wikiделам вместе с мёрджем. label Oct 15, 2024
UrPrice
UrPrice reviewed Oct 15, 2024
View reviewed changes
Resources/Prototypes/SS220/Entities/Clothing/Head/hats.yml
@@ -166,7 +166,7 @@
- type: entity
parent: ClothingHeadBase
id: ClothingHeadHatSeniorService
name: фуражка ведущего сервисного отдела
name: фуражка сервисного администратора #SS220 rename
Copy link
Collaborator

Choose a reason for hiding this comment

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

а для чего тут #ss220 rename ?

Copy link
Author

Choose a reason for hiding this comment

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

Чтоб знали

UrPrice
UrPrice approved these changes Oct 15, 2024
View reviewed changes
Copy link
Collaborator

@UrPrice UrPrice left a comment

Choose a reason for hiding this comment

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

by Ady4ik (устно апрув получили)

@Tris0r
Copy link
Author

Tris0r commented Oct 15, 2024

Победа!

@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 15, 2024

Ну ждем вики тогда

@Tris0r
Copy link
Author

Tris0r commented Oct 15, 2024

Ну ждем вики тогда

А когда на Вики менять?

@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 15, 2024

А когда на Вики менять?

хз, но поставили же такой тег
image

@Tris0r
Copy link
Author

Tris0r commented Oct 15, 2024

А когда на Вики менять?

хз, но поставили же такой тег
image

Бубенка говорит перед добавлением в игру, чтобы не было путаницы.

@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 15, 2024

Бубенка говорит перед добавлением в игру, чтобы не было путаницы.

Ну, тогда получается прадокс)))

@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 15, 2024

В целом ПР одобрен же, так что считай что сейчас это и есть "перед добавлением в игру"

@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 15, 2024

Или имелось ввиду перед обновлением серверов?

@Tris0r
Copy link
Author

Tris0r commented Oct 15, 2024

Или имелось ввиду перед обновлением серверов?

я хрен его знает.

@Tris0r
Copy link
Author

Tris0r commented Oct 15, 2024

Или имелось ввиду перед обновлением серверов?

он говорит перед обновлением.

@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 15, 2024

Ну не забудьте тогда

@Kirus59 Kirus59 removed the Need to add Wiki220 Требуется добавить на Wiki/поставить задачу Wikiделам вместе с мёрджем. label Oct 15, 2024
@Kirus59 Kirus59 merged commit cf806dd into SerbiaStrong-220:master Oct 15, 2024
25 checks passed
@Tris0r
Copy link
Author

Tris0r commented Oct 15, 2024

Ну не забудьте тогда

Ради тебя специально не забуду.

@UrPrice
Copy link
Collaborator

UrPrice commented Oct 16, 2024

Ну ждем вики тогда

Поправил описание лейбла, чтобы не было замешательств

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kirus59 Kirus59 Kirus59 left review comments

@UrPrice UrPrice UrPrice approved these changes

@Ady4ik Ady4ik Awaiting requested review from Ady4ik

Assignees

@Kirus59 Kirus59

Labels
Changes: Localization Изменение затронуло файлы ".ftl" Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Tris0r @Kirus59 @UrPrice