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

Syndie jaws of life fix of speed modifier #1842

Merged
merged 4 commits into from
Sep 16, 2024

Conversation

lexaSvarshik
Copy link

@lexaSvarshik lexaSvarshik commented Sep 11, 2024

Описание PR

Фикс того, что челюсти жизни синдиката медленнее обычных челюстей. Теперь челюсти синдиката открывают запитанные двери в три раза быстрее. Скорость обычных челюстей снова нормальная.
На основе трекера https://discord.com/channels/1097181193939730453/1283096304276340838

Проверки

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

Изменения

🆑

  • tweak: Скорость открытия запитанных дверей теперь одинакова для обеих челюстей жизни и составляет 11 секунд.

Фикс того, что челюсти жизни синдиката медленнее обычных. Теперь челюсти синдиката открывают запитанные двери в три раза быстрее.
@github-actions github-actions bot added the Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" label Sep 11, 2024
@Kirus59
Copy link
Collaborator

Kirus59 commented Sep 11, 2024

В проверках нужно так ставить [X] (ну или [x]) чтобы галочка была.

@DexlerXD
Copy link

исправить бы описание пра, а именно:

  1. видео перенести в медиа
  2. написать галочки, как отметил Кирус
  3. написать чейнджлог по форме (в комментариях пример есть)

Copy link

@gogenych gogenych left a comment

Choose a reason for hiding this comment

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

Спасибо что попросили ревью, потому что тут явно что-то не так. Я порылся в комитах и обнаружил, что наш пр #624 наложился на пару апстримов.

Что нужно сделать:

  1. Все челюсти должны быть с модификатором скорости "3" т.к. им изначально задавалась такая скорость если у них появится возможность взламывать пол. Она у них появилась, но обычным скорость вернуть забыли.

  2. Поправить комментарии к изменениям. Каждое новое изменение должно идти под своим комментарием, а тут три история из минимум трех правок под комментарием Адуча полугодичной давности.

@lexaSvarshik
Copy link
Author

То есть у нас синди-челюсти будут идентичны обычным? ПР поправлю на досуге

@gogenych
Copy link

То есть у нас синди-челюсти будут идентичны обычным? ПР поправлю на досуге

сейчас у синди челюстей 11 секунд скорость. Она такая потому, что когда-то им дали возможность ломать пол и таким образом забалансили. Чуть позже оффы дали обычным челюстям возможность ломать пол тоже и мы должны забалалансить обычные до 11 секунд

@lexaSvarshik
Copy link
Author

Тогда, если делать обеим челюстям 11 секунд на открытие запитанной двери, модификатор скорости нужно будет поставить на единицу. И тогда у нас будут синди-челюсти абсолютно идентичны обычным. Так и задумано, что предмет из аплинка за 2тк абсолютно идентичен предмету, который изучается в рнд через 40 минут с начала смены?

@gogenych
Copy link

Так и задумано, что предмет из аплинка за 2тк абсолютно идентичен предмету, который изучается в рнд через 40 минут с начала смены?

да

lexaSvarshik and others added 3 commits September 15, 2024 21:12
Теперь скорость взлома двери одинаковая для обоих челюстей, составляет 11 секунд
@lexaSvarshik
Copy link
Author

Довёл до ума описание ПР'а, фикс в коммите. Модификатор я поставил на 1, т.к. именно при нём скорость открытия будет составлять 11 секунд

@DexlerXD DexlerXD requested a review from gogenych September 15, 2024 18:28
Copy link

@DexlerXD DexlerXD left a comment

Choose a reason for hiding this comment

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

технически норм

@Ady4ik Ady4ik merged commit ce4aa61 into SerbiaStrong-220:master Sep 16, 2024
18 checks passed
@lexaSvarshik lexaSvarshik deleted the Jaws-of-life-rebalance branch October 9, 2024 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants