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

Ss220 uplink minor fixes and discounts #1968

Conversation

lexaSvarshik
Copy link

Описание PR

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

  • Изменен нижний порог скидки метательных ножей
  • Изменен нижний порог скидки импланта "свобода"
  • Добавлены скидки для: миниарбалета, двойного энергомеча, майндслейва, термалов, импланта термалов, сегвея, набора улучшения мима. Скидки сделаны аналогично оффовским. Нижний порог скидок на даблу и арбалет сделан нескольким выше ввиду баланса.

Проверки

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

Изменения

no cl no fun

В обновлении с уменьшением цен на мало используемые предметы из аплинка цену снизили, в связи с чем снижаю нижний порог скидки
@github-actions github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" labels Sep 29, 2024
@DexlerXD DexlerXD requested a review from Ady4ik September 29, 2024 23:16
@DexlerXD DexlerXD requested a review from gogenych September 29, 2024 23:17
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.

Первый раз слышу о этих изменениях. Баланс должен обсуждаться, а не просто заливаться на гит "потому что логично". У нас уже есть опыт подобных правок, пол года разгребаем

@lexaSvarshik
Copy link
Author

Первый раз слышу о этих изменениях.

Порог скидок изменен для предметов из обновы #SS220-underused-uplinks-stuff. Им цену скинули, но у оффов другая цена и в итоге там скидка могла составлять 0%.

@lexaSvarshik
Copy link
Author

Ну а скидки нашим предметам по типу термалов сделаны, потому что скидок нет... А должны быть...

Copy link

@Ady4ik Ady4ik left a comment

Choose a reason for hiding this comment

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

Dnm, требует обсуждения.

@Surani1
Copy link

Surani1 commented Oct 5, 2024

Dnm, требует обсуждения.

Завтра на планёрке?

@Ady4ik Ady4ik self-requested a review October 6, 2024 13:31
@DexlerXD DexlerXD merged commit 43c48be into SerbiaStrong-220:master Oct 6, 2024
21 of 23 checks passed
@lexaSvarshik lexaSvarshik deleted the ss220-uplink-minor-fixes-and-discounts branch October 9, 2024 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants