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

Add: Job Brig Physician #1644

Closed
wants to merge 3 commits into from
Closed

Conversation

Glamyrio
Copy link

@Glamyrio Glamyrio commented Nov 6, 2024

Что этот PR делает

Добавляет относительно новую профессию "Бриг Медик" и так же место на карте для профессии, но очень много не модульного, к примеру оутфит плазмамужика в модуле не работает, а айди карта в code/game/objects/item/weapons/cards_ids нужна чтобы было нормальное название card skins у консоли ГП, а иконка пда не модульная, чтобы ПДАпейнтер работало корректно. Потребуется обновление вики и нанокарт.

Почему это хорошо для игры

Новая профессия для игры, почему нет?

Изображения изменений

бригпедик12
бригпедик13
бригпедик11
бригпедик10
бригпедик8
бригпедик7
бригпедик6
бригпедик5
бригпедик4
бригпедик9
бригпедик3
бригпедик2
бригпедик1

Тестирование

Проверял на локалке. Всё работает см. изображения выше. Придётся ещё некоторое время подержать в тме для выявления ошибок.

Changelog

🆑 Glamyr
add: Добавлена новая профессия в отдел службы безопасности. (подробности в Пулл Реквесте)
add: Добавлена форма-водолазка и юбка-водолазка для данной профессии.
add: Добавлен медпункт отдела службы безопасности во все наши карты.
/:cl:

@github-actions github-actions bot added 💾 Изменение конфига Ф-ф-фуриор...? :feelsgood: Частичная модульность Не всегда получается всё впихнуть в модуль, увы. 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! Кибериада Этот ПР изменил что-то на Кибериаде (BoxStation) Керберос Этот ПР изменил что-то на Керберосе (DeltaStation) Цереброн Этот ПР изменил что-то на Цереброне (MetaStation) labels Nov 6, 2024
@Glamyrio
Copy link
Author

Glamyrio commented Nov 6, 2024

Рецензенты же добавят теги, правда же? Правда?

@AyIong
Copy link
Collaborator

AyIong commented Nov 6, 2024

Уже много раз обсуждалось почему мы НЕ будет возвращать бригмеда, много раз это обжёвывалось, и ты решил, ни с кем не поговорив, и ДАЖЕ никого не уведомив, вернуть его? Очень не красиво, и опрометчиво, так как потраченные силы и время, скорее всего пойдут на смарку

К тому же, апстрим уже выпиливал бригмеда, взамен дав 1 слот в СБ. Ты же возвращаешь его, не забирая этот самый слот, в итоге, мало того что у СБ появляется надёжный врач и отпадает необходимость зашиваться в меде, из-за чего происходит баф СБ

И ещё, иконки заменяются модульно, перезаписыванием пути иконки у определённого предмета.
В идеале, перед тем как что-то делать, тебе надо было спросить у нас, или у челиков с апстрима, возмооожно, они бы согласились на бригмеда, и не пришлось бы делать модульно

Я лично против его возврата

@AyIong AyIong added the Do Not Merge Не заливать. В ожидании хэда/хоста. Или просто временный тестмерж. label Nov 6, 2024
@Glamyrio
Copy link
Author

Glamyrio commented Nov 6, 2024

Уже много раз обсуждалось почему мы НЕ будет возвращать бригмеда, много раз это обжёвывалось, и ты решил, ни с кем не поговорив, и ДАЖЕ никого не уведомив, вернуть его? Очень не красиво, и опрометчиво, так как потраченные силы и время, скорее всего пойдут на смарку

К тому же, апстрим уже выпиливал бригмеда, взамен дав 1 слот в СБ. Ты же возвращаешь его, не забирая этот самый слот, в итоге, мало того что у СБ появляется надёжный врач и отпадает необходимость зашиваться в меде, из-за чего происходит баф СБ

И ещё, иконки заменяются модульно, перезаписыванием пути иконки у определённого предмета. В идеале, перед тем как что-то делать, тебе надо было спросить у нас, или у челиков с апстрима, возмооожно, они бы согласились на бригмеда, и не пришлось бы делать модульно

Я лично против его возврата

понял тебя, таки получается что все старания на смарку, к тому же не был уведомлён о вырезании бриг медика на этом билде. Ну, получается всё

@m-dzianishchyts
Copy link
Collaborator

Нет обсуждения о повторном вводе бригмеда.
Ветка пусть остается, вернемся к этому ПРу, когда обсуждение возникнет и будет одобрение хотя бы одного старшего разработчика

@AyIong
Copy link
Collaborator

AyIong commented Nov 6, 2024

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge Не заливать. В ожидании хэда/хоста. Или просто временный тестмерж. :feelsgood: Частичная модульность Не всегда получается всё впихнуть в модуль, увы. 💾 Изменение конфига Ф-ф-фуриор...? 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден Керберос Этот ПР изменил что-то на Керберосе (DeltaStation) Кибериада Этот ПР изменил что-то на Кибериаде (BoxStation) Цереброн Этот ПР изменил что-то на Цереброне (MetaStation)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants