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

bugfix: you can no longer be immortal with sindie uno card #6038

Merged
merged 3 commits into from
Oct 16, 2024

Conversation

NightDawnFox
Copy link
Contributor

Описание

Ссылка на предложение/Причина создания ПР

только для админов
Обычное

Тесты

запустил локалку
пострелял по парню в мехе. Урон прошел, карта упала (пойдет)
пострелял по парню, у которого антидроп. Урон прошёл
пострелял по парню, антидроп у меня. Урон прошёл.
пострелял без антидропа - всё хорошо

@NightDawnFox NightDawnFox changed the title увыы bugfix: you can no longer be immortal with sindie uno card Oct 16, 2024
Copy link
Member

@Daeberdir Daeberdir left a comment

Choose a reason for hiding this comment

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

Просто возвращай результат switcharoo, нету смысла по несколько раз заставлять людей что-то бросать. FORCE = TRUE может быть вариантом для дропа и подьёма внутри прока, но оно сейчас плохо работает с имплантами тулсетов. В любом случае, нужно возвращать пушку в руку стрелка, если не получилось бросить карту.

Copy link
Member

@Daeberdir Daeberdir left a comment

Choose a reason for hiding this comment

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

Окей. Должно решить проблему.

@Daeberdir Daeberdir added Critical Priority Мы облажались Has Approve Have been approved by somebody from Review Team and awaiting for one more approve labels Oct 16, 2024
Copy link
Contributor

@Den109G Den109G left a comment

Choose a reason for hiding this comment

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

Фиксбаг.

@Den109G Den109G added this pull request to the merge queue Oct 16, 2024
Merged via the queue into ss220-space:master220 with commit e2294a1 Oct 16, 2024
12 checks passed
@Daeberdir Daeberdir removed Critical Priority Мы облажались Has Approve Have been approved by somebody from Review Team and awaiting for one more approve labels Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants