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

return machine upgrade system #9

Closed
wants to merge 7 commits into from

Conversation

KillanGenifer
Copy link

Возвращает систему улучшения машин, технологию, открывающую компоненты и РПЗД.

Основная часть кода взята с фронтира.
За корректную работу системы замены компонентов вручную и РПЗД не ручаюсь, так как не осилил код. Остальное, в виде ивентов обработки компонентов машины после замены запчастей была осмотрена и переделана индивидуально под нашу сборку.

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

Если PR будет мёржнут, то докину дополнительные улучшения для новых машин, по типу упаковщика, плиточника, карабельных пушек и т.п.

goool

@KillanGenifer KillanGenifer requested a review from Morb0 as a code owner October 25, 2024 16:18
@lzk228
Copy link
Collaborator

lzk228 commented Oct 25, 2024

а есть кнопка unaprove?

@KillanGenifer
Copy link
Author

а есть кнопка unaprove?

Какая? Где?

@AwareFoxy
Copy link
Collaborator

Систему улучшений убрали не просто так.
space-wizards/space-station-14#21055

Copy link
Collaborator

@lzk228 lzk228 left a comment

Choose a reason for hiding this comment

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

кажется ты забыл заполнить часть "почему/баланс" в описании твоего пул реквеста, зпаолни пж

Copy link
Collaborator

@AwareFoxy AwareFoxy left a comment

Choose a reason for hiding this comment

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

В ПРе ни слова о том, почему вернуть эту механику - хорошая идея.

@KillanGenifer
Copy link
Author

В целом позиция оффов мне ясна, тогда фиг с ним.

FireNameFN pushed a commit to FireNameFN/space-station-14-next that referenced this pull request Jan 12, 2025
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.

3 participants