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

removes stock market #11083

Merged

Conversation

LudwigVonChesterfield
Copy link
Contributor

@LudwigVonChesterfield LudwigVonChesterfield commented Mar 20, 2023

Описание изменений

По началу я хотел просто занерфить рынок акций до уровня "зачем это юзать" (#10999) но оставить как флаворовую фичу, но потом я посмотрел код рынка... мне стало дурно, и я подумал "О, может на ТГ уже решили мою проблему", и не ошибся! Действительно решили: tgstation/tgstation#37414

Меня это заставило задуматься, и...

Этот ПР выпиливает компьютер рынка акций из карго, и всё связанное с ним (кроме спрайтов?).

Я не против чтобы его вернули если кто-то сделает код красивым, а рынок акций менее прибыльным)

Почему и что этот ПР улучшит

  1. компьютер акций преумножает кредиты карго, что усугубляет любой абуз их денег в разы делая из маленькой проблемы большую
  2. компьютер акций никак не поощряет взаимодействие карго с кем-либо на станции, а геймплей у него очень унылый и вредный по отношению к игроку (Refresh -> Refresh -> Refresh -> ЗЕЛЁНЕНЬКАЯ СТРЕЛОЧКА КРАСНЕНЬКАЯ ПОРА ВЫВОДИТЬ!!!). скиннер бокс без сути какой-то
  3. код некрасивый и нерасширяемый, но его много что делает рефактор затруднительным

Чеинжлог

🆑 Luduk

  • rscdel: Удалён рынок акций.

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@volas
Copy link
Member

volas commented Mar 20, 2023

Комитет по сохранению фич воздержится от защиты этой фичи

@KIBORG04
Copy link
Member

Не против

@LudwigVonChesterfield LudwigVonChesterfield added Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже Experimental labels Mar 20, 2023
Copy link
Member

@volas volas left a comment

Choose a reason for hiding this comment

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

можешь мержить, если решишь что ок

@LudwigVonChesterfield
Copy link
Contributor Author

предпочту помариновать недели две чтобы узнать как повлияет на баланс режимов

@Ro2tCrab
Copy link
Contributor

предпочту помариновать недели две чтобы узнать как повлияет на баланс режимов

К чёрту фичу.
Давай на плаху!
Можно потом и стоимость скрапа подкрутить.

@LudwigVonChesterfield
Copy link
Contributor Author

На режимы как оказалось не сильно повлияло.

@LudwigVonChesterfield LudwigVonChesterfield merged commit b757703 into TauCetiStation:master Apr 1, 2023
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Apr 1, 2023
@Deahaka
Copy link
Contributor

Deahaka commented Nov 14, 2024

Комитет по сохранению фич воздержится от защиты этой фичи

what a shame...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Experimental Map Edit Revert / Removal Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants