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

Teleporter grid restriction #722

Merged

Conversation

SkaldetSkaeg
Copy link

@SkaldetSkaeg SkaldetSkaeg commented Feb 8, 2024

Описание PR

При попытке создать второй портал на другом гриде первый портал теперь будет удаляться.
Медиа

image

Проверки

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

Изменения

  • tweak: При попытке разместить второй портал в другом гриде теперь первый портал удаляется.

@github-actions github-actions bot added the Changes: Localization Изменение затронуло файлы ".ftl" label Feb 8, 2024
@SkaldetSkaeg
Copy link
Author

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

@Kit0vras
Copy link

Kit0vras commented Feb 8, 2024

Желательно не на любом другом гриде, а именно на гриде с трейд-постом, разве нет? Хотя тут наверно лучше Адуча спросить
@Ady4ik

@SkaldetSkaeg
Copy link
Author

Желательно не на любом другом гриде, а именно на гриде с трейд-постом, разве нет? Хотя тут наверно лучше Адуча спросить @Ady4ik

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

@Kit0vras
Copy link

Kit0vras commented Feb 8, 2024

Ладно, ты прав, не подумал о таком

@DexlerXD
Copy link

DexlerXD commented Feb 9, 2024

@Ady4ik одобряем в итоге?

@VladIsLove-e
Copy link

А от радиуса не получится?

@VladIsLove-e
Copy link

Это было бы логичнее

@Ady4ik
Copy link

Ady4ik commented Feb 9, 2024

Давайте пока рестрикт по гриду, это вполне себе логично, потом может придумаем, как сделать грамотную проверку на расстояние

@@ -1,4 +1,4 @@
using Content.Shared.DoAfter;
using Content.Shared.DoAfter;
Copy link

Choose a reason for hiding this comment

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

нервный тик в глазу

Copy link
Author

Choose a reason for hiding this comment

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

Я хз, есть идеи как это менять?
ctrl+c ctrl+v не фиксит

Copy link

Choose a reason for hiding this comment

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

Я хз, есть идеи как это менять? ctrl+c ctrl+v не фиксит

берешь оригинальный файл, вставляешь с заменой и через гит отменяешь все изменения, кроме этого

Copy link

Choose a reason for hiding this comment

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

а все уже, хорош

@DexlerXD DexlerXD merged commit 6dc4a4d into SerbiaStrong-220:master Feb 9, 2024
16 checks passed
@SkaldetSkaeg SkaldetSkaeg deleted the Teleporter_grid_restriction branch February 12, 2024 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение затронуло файлы ".ftl"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants