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

Новая роль - Пилот Сб #1978

Merged
merged 5 commits into from
Apr 8, 2024
Merged

Новая роль - Пилот Сб #1978

merged 5 commits into from
Apr 8, 2024

Conversation

poeMota
Copy link
Contributor

@poeMota poeMota commented Mar 19, 2024

Описание PR

Медиа

image

Проверки

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

Изменения

🆑

  • 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: 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 Mar 19, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Mar 19, 2024 •
edited
Loading

RSI Diff Bot; head commit 485fea1 merging into 29a1716
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
Pilot Added

Resources/Textures/Objects/Misc/id_cards.rsi

State Old New Status
idpilot Added

Edit: diff updated after 485fea1

@lzk228
Copy link
Collaborator

lzk228 commented Mar 19, 2024

возьми иконку с коммита
07830ff

@poeMota
Copy link
Contributor Author

poeMota commented Mar 19, 2024

возьми иконку с коммита 07830ff

Ты автор?

@poeMota
Copy link
Contributor Author

poeMota commented Mar 19, 2024

я чуть подредактирую её всё равно, обводку как у инспектора сб сделаю

@lzk228
Copy link
Collaborator

lzk228 commented Mar 19, 2024

я в дс потом кину копирайт

@Nimfar11
Copy link
Contributor

Думаю стоит добавить на карты Корвакса, пилота часто не хватает. И вечно споры с ним, что корабли есть, а водить его может только ГСБ у которого обычно нет времени обучить офицера пилотированию.

@lzk228
Copy link
Collaborator

lzk228 commented Mar 20, 2024

Думаю стоит удалить корабли с карт Корвакса.

@poeMota
Copy link
Contributor Author

poeMota commented Mar 20, 2024

Думаю стоит удалить корабли с карт Корвакса.

пахах, если бы ты знал по какому поводу я сделал этот ПР 💀

@Nimfar11
Copy link
Contributor

Nimfar11 commented Mar 20, 2024 •
edited
Loading

Думаю стоит удалить корабли с карт Корвакса.

Зачем удалять контент? Они есть, их можно использовать, это даёт больше вариативности для СБ и антагов.

К тому же оффы потенциально хотят добавить ивенты с появлением разных шаттлов в космосе. Потому можно заранее протестить роль и правила для неё.

CatBackGround
CatBackGround approved these changes Mar 21, 2024
View reviewed changes
Copy link
Contributor

@CatBackGround CatBackGround left a comment

Choose a reason for hiding this comment

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

Отсмотрел, всё цивильно и без проёбов.

JerryImMouse
JerryImMouse approved these changes Mar 22, 2024
View reviewed changes
Copy link
Contributor

@JerryImMouse JerryImMouse left a comment

Choose a reason for hiding this comment

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

Отсмотрел, всё цивильно и без проёбов.

@CatBackGround
Copy link
Contributor

Как только вернусь с поездки - перенесу на свою локалку и начну проводить глубокий анализ. А так на бумаге все заебись.

@Syxapik
Copy link
Contributor

Syxapik commented Mar 26, 2024

Как только вернусь с поездки - перенесу на свою локалку и начну проводить глубокий анализ. А так на бумаге все заебись.

Глубоки анализ.. ты кто вообще такой? Ревизор от тайного правительства? 💀

@CatBackGround
Copy link
Contributor

CatBackGround commented Mar 26, 2024 •
edited
Loading

Глубоки анализ.. ты кто вообще такой? Ревизор от тайного правительства? 💀

Пхах, спасибо за новую идею для своего лора.
А так - я всего лишь самоучка на YAML, C# и Fluent. Я и работоспособность проверю, и опыта немного по YAML получу.

P.S. И может быть продумаю свою отыгровку на нём, если его одобрят и сольют в master.

@Morb0 Morb0 merged commit 11cc48c into space-syndicate:master Apr 8, 2024
16 checks passed
@lzk228
Copy link
Collaborator

lzk228 commented Apr 8, 2024

морти... зачем....

@CatBackGround
Copy link
Contributor

Да ладно...
Свершилось.

@SoulFN
Copy link
Contributor

SoulFN commented Apr 9, 2024

Победа

@poeMota poeMota deleted the pilot branch May 16, 2024 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Syxapik Syxapik Syxapik left review comments

@lzk228 lzk228 lzk228 left review comments

@Deserty0 Deserty0 Deserty0 left review comments

@CatBackGround CatBackGround CatBackGround approved these changes

@JerryImMouse JerryImMouse JerryImMouse approved these changes

@ficcialfaint ficcialfaint Awaiting requested review from ficcialfaint ficcialfaint is a code owner

@Morb0 Morb0 Awaiting requested review from Morb0 Morb0 is a code owner

Assignees
No one assigned
Labels
Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation 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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@poeMota @lzk228 @Nimfar11 @CatBackGround @Syxapik @SoulFN @JerryImMouse @Deserty0 @Morb0 @KonChenY1