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

FIX: Nuke unanchor w/o disk #2201

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ReeZer2
Copy link

@ReeZer2 ReeZer2 commented Nov 3, 2024

Описание PR

Ядерку теперь можно открепить без диска ядерной аутентификации (fix: #2194)

Медиа

Content Client_pZqwEeEwUJ
Content Client_rk3aeq3Eih

Проверки

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

Изменения

🆑 ReeZii

  • fix: Ядерную бомбу теперь снова можно открепить без диска ядерной аутентификации

@github-actions github-actions bot added the Changes: C# Изменение затронуло файлы ".cs" label Nov 3, 2024
@UrPrice UrPrice requested a review from Ady4ik November 3, 2024 09:06
@lexaSvarshik
Copy link

Это очень спорно убирать болты чисто из-за вора, поскольку в режиме нюки станционную ядерку могут просто спрятать где-то в техах, чтоб ЯО её вовек не нашли.
Или классический сценарий - какие-нибудь утиля штурмуют шаттл ЯО и воруют оттуда их ядерку.

@Kirus59
Copy link
Collaborator

Kirus59 commented Nov 4, 2024

Это очень спорно убирать болты чисто из-за вора, поскольку в режиме нюки станционную ядерку могут просто спрятать где-то в техах, чтоб ЯО её вовек не нашли. Или классический сценарий - какие-нибудь утиля штурмуют шаттл ЯО и воруют оттуда их ядерку.

Так у нюкеров все равно нет кода от станционной ядерки, у них же своя.

@lexaSvarshik
Copy link

Это очень спорно убирать болты чисто из-за вора, поскольку в режиме нюки станционную ядерку могут просто спрятать где-то в техах, чтоб ЯО её вовек не нашли. Или классический сценарий - какие-нибудь утиля штурмуют шаттл ЯО и воруют оттуда их ядерку.

Так у нюкеров все равно нет кода от станционной ядерки, у них же своя.

Как это нету? У них код либо от своей, либо от станционной, там когда как.

@Kirus59
Copy link
Collaborator

Kirus59 commented Nov 4, 2024

Как это нету? У них код либо от своей, либо от станционной, там когда как.

А да, оффы до сих пор это не исправили, там спавнится код на рандомную ядерку на сервере, не думаю что это должно так работать.

@Ady4ik
Copy link

Ady4ik commented Nov 5, 2024

Это очень спорно убирать болты чисто из-за вора, поскольку в режиме нюки станционную ядерку могут просто спрятать где-то в техах, чтоб ЯО её вовек не нашли.
Или классический сценарий - какие-нибудь утиля штурмуют шаттл ЯО и воруют оттуда их ядерку.

Вот тут поддерживаю, очень странное изменение ради вора. Дайте ему просто уникальный инструмент, отмычку ту же и только при помощи нее можно будет открутить. А потом с отмычкой уже можно будет что нибудь интересное мутить.

@github-actions github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" labels Nov 11, 2024
@ReeZer2 ReeZer2 marked this pull request as draft November 11, 2024 17:21
@ReeZer2
Copy link
Author

ReeZer2 commented Nov 11, 2024

@Ady4ik пока добавил функционал чисто для нюки, во вторых нужен спрайт. Ну и обсудить как-то детальнее, для чего именно будет нужна отмычка и ее функционал

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Dec 1, 2024
Copy link

github-actions bot commented Dec 1, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Merge Conflict Some content has conflicts. Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Боеголовку не унести без диска
4 participants