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

АПИИ 3 #52

Merged
merged 10 commits into from
Nov 20, 2024
Merged

АПИИ 3 #52

merged 10 commits into from
Nov 20, 2024

Conversation

FireNameFN
Copy link
Collaborator

Описание PR

Поправил некоторые баги, которые были от компонентов вора. Теперь АПИИ даётся имплант хранилища, в котором содержится его начальное снаряжение. Добавлен глушитель связи.

Ссылка на ветку

https://discord.com/channels/919301044784226385/1303844533695414353

Список изменений

🆑

  • tweak: Теперь у АПИИ сразу появляется имплант хранилища с вещами в нём.
  • tweak: Теперь у АПИИ есть глушитель связи.

@FireNameFN FireNameFN requested a review from Morb0 as a code owner November 14, 2024 08:00
@github-actions github-actions bot added Changes: Localization Изменяет локализацию Status: Needs Review labels Nov 14, 2024
Copy link
Collaborator

@Tornado-Technology Tornado-Technology left a comment

Choose a reason for hiding this comment

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

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

Content.Server/Thief/Systems/ThiefBeaconSystem.cs Outdated Show resolved Hide resolved
@FireNameFN
Copy link
Collaborator Author

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

Сказал "сойдёт" и нажал чендж реквест?

@CaptainSqrBeard
Copy link

CaptainSqrBeard commented Nov 17, 2024

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

Что зачёркнуто - то уже неактуально

Разное

  • Зависимость от карты: На некоторых картах замапленны интеллкарты, в том числе они могут быть замапленны прям на спутнике ИИ. На тушкане вообще ИИ в телекоммах сидит
  • АПИИ правильно показывается в манифесте раунда, но не в манифесте игроков. Там он показывается как простой вор
  • АПИИ не может захватить ИИ без игрока
  • У украденной ИИ есть доступ к двоичному каналу. Не уверен, проблема это или нет.

Снаряжение

  • Тулбокс АПИИ невозможно отличить от тулбокса вора в спавн меню
  • Маяки и тулбоксы АПИИ и вора в целом внешне идентичны (как раз работа для спрайтеров :trollface:)
  • У игрока нету выбора между наборами в тулбоксе, при этом сами наборы схожи наборам вора. Было-бы неплохо подумать над новыми наборами и каким-то разнообразием выбора.

api_diff

@FireNameFN
Copy link
Collaborator Author

АПИИ не может захватить ИИ без игрока

АПИИ не будет спавниться, когда нет ЦИИ.

У украденной ИИ есть доступ к двоичному каналу. Не уверен, проблема это или нет.

У АПИИ есть глушилка.

У игрока нету выбора между наборами в тулбоксе, при этом сами наборы схожи наборам вора. Было-бы неплохо подумать над новыми наборами и каким-то разнообразием выбора.

Тулбокс его будет ещё переработан.

@AwareFoxy AwareFoxy merged commit a4b1611 into space-syndicate:master Nov 20, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменяет локализацию Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants