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

RnD Balance revert #2092

Merged
merged 3 commits into from
Oct 17, 2024
Merged

Conversation

UrPrice
Copy link
Collaborator

@UrPrice UrPrice commented Oct 15, 2024

Описание PR

Ревёртим изменения следующего баланса #496 и вносим правку в том, что:

  • Аномалии (объекты с компонентой Anomally) больше не коллапсируют при попытке пройти через портал
  • Аномалии (объекты с компонентой Anomally) более не могут пользоваться порталом

Мне нужна проверка и помощь в реализации, если где-то были допущены ошибки.
Как минимум сейчас, из несущественных, но всё же упущений - Попап уведомление возникает слишком часто.

Медиа
https://i.imgur.com/hS4z7jU.gif

Проверки

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

Изменения
🆑

  • tweak: аномалии (и их носители) более не коллапсируют при попытке пройти через портал, как и не проходят через него вовсе.

@github-actions github-actions bot added the Changes: Localization Изменение затронуло файлы ".ftl" label Oct 15, 2024
@Kirus59 Kirus59 self-assigned this Oct 15, 2024
Content.Shared/Teleportation/Systems/SharedPortalSystem.cs Outdated Show resolved Hide resolved
Resources/Locale/en-US/portal/portal.ftl Outdated Show resolved Hide resolved
Resources/Locale/ru-RU/portal/portal.ftl Outdated Show resolved Hide resolved
@UrPrice
Copy link
Collaborator Author

UrPrice commented Oct 16, 2024

@Kirus59 Срёт ошибками клиентскими при наложении аномалии на портал. Пока ещё не понял с чем связано, но будто бы по хорошему поправить..

@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 16, 2024

@Kirus59 Срёт ошибками клиентскими при наложении аномалии на портал. Пока ещё не понял с чем связано, но будто бы по хорошему поправить..

Скрин ошибки бы...

@UrPrice
Copy link
Collaborator Author

UrPrice commented Oct 16, 2024

@Kirus59 Срёт ошибками клиентскими при наложении аномалии на портал. Пока ещё не понял с чем связано, но будто бы по хорошему поправить..

Скрин ошибки бы...

Забей, она не с этим связана, а с тем, что аномалию берут силовой пушкой:
[ERRO] system.joint: While adding joint tether to entity аномалия (2427/n2486, AnomalyBluespace), the joint already existed for the connected entity (2475/n2507, TetherEntity).

Это можно проверять и мерджить

@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 16, 2024

@UrPrice Спам попапов удалось решить?

@UrPrice
Copy link
Collaborator Author

UrPrice commented Oct 16, 2024

@UrPrice Спам попапов удалось решить?

неа, где-то до х12 попапов за раз может появиться, но не бесконечный спам

@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 16, 2024

неа, где-то до х12 попапов за раз может появиться, но не бесконечный спам

Тогда мб сделаешь, чтобы у кукол не возникал этот попап, что думаешь?
Просто видимо несколько раз вызывается этот метод пока кукла в портале.....

@UrPrice
Copy link
Collaborator Author

UrPrice commented Oct 16, 2024

неа, где-то до х12 попапов за раз может появиться, но не бесконечный спам

Тогда мб сделаешь, чтобы у кукол не возникал этот попап, что думаешь? Просто видимо несколько раз вызывается этот метод пока кукла в портале.....

Тогда неинтуитивно, почему портал не срабатывает на человвека.

@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 16, 2024

А, это же шейрд, спам такой скорее всего видит только тот, кто заходит в портал (поэтому с аномалией и нет его)

@UrPrice
Copy link
Collaborator Author

UrPrice commented Oct 16, 2024

А, это же шейрд, спам такой скорее всего видит только тот, кто заходит в портал (поэтому с аномалией и нет его)

Да, проверил, там проспам только на клиенте кто пытается войти. В принципе не критично. Для остальных проспама нет.

@Kirus59 Kirus59 merged commit 5f34f07 into SerbiaStrong-220:master Oct 17, 2024
16 checks passed
@UrPrice UrPrice deleted the AnomalyNormalize branch October 22, 2024 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение затронуло файлы ".ftl" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants