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

Economy for ADT #222

Merged
merged 9 commits into from
Aug 12, 2024
Merged

Economy for ADT #222

merged 9 commits into from
Aug 12, 2024

Conversation

KashRas2
Copy link
Collaborator

@KashRas2 KashRas2 commented Aug 12, 2024

Описание PR

Попытка адаптации данного пулл реквеста под наши реалии - Вот тут.
Это не значит, что я присваиваю себе авторство этого кода как себе, так и нашему проекту (это больше для простых обывателей, что наткнуться на нашу сборку).
К слову. Пулл реквест был взят до обновления readme.md у первоисточника (на момент написания readme всё ещё не изменен.)

Медиа
Вероятно тут что-то будет, как я доведу всё до ума.

Проверки

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

Изменения

🆑 KashRas2

  • add: Была добавлена система экономики, которая в значительной степени разнообразит геймплей.
  • tweak: Станционный бюджет теперь накапливается на особую карту, с которой можно как вывести, так и внести деньги.

@KashRas2 KashRas2 marked this pull request as draft August 12, 2024 12:57
@github-actions github-actions bot added Changes: UI Изменение интерфейса Changes: Sprites Изменение спрайтов Changes: Localization Изменение локализации labels Aug 12, 2024
Copy link
Contributor

github-actions bot commented Aug 12, 2024

RSI Diff Bot; head commit 2684fe5 merging into bf22491
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/ADT/Objects/Devices/eftpos.rsi

State Old New Status
eftpos-screen Added
eftpos Added

Resources/Textures/ADT/Objects/Misc/cards.rsi

State Old New Status
budgetcard Added
car_budget Added
civ_budget Added
eng_budget Added
med_budget Added
sci_budget Added
sec_budget Added
srv_budget Added

Resources/Textures/ADT/Structures/Machines/atm.rsi

State Old New Status
off Added
on Added

Edit: diff updated after 2684fe5

@KashRas2 KashRas2 marked this pull request as ready for review August 12, 2024 13:19
@KashRas2
Copy link
Collaborator Author

Временно уберу с драфта, так как мне нужны тесты.

Copy link
Collaborator

@Schrodinger71 Schrodinger71 left a comment

Choose a reason for hiding this comment

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

Так, ну в принципе всё просмотрел, с кодом ок, прототипы и локализация, тоже вроде всё окей

@Schrodinger71 Schrodinger71 added this pull request to the merge queue Aug 12, 2024
Merged via the queue into AdventureTimeSS14:master with commit 54f4a4b Aug 12, 2024
15 checks passed
@KashRas2 KashRas2 deleted the economy branch November 19, 2024 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение локализации Changes: Sprites Изменение спрайтов Changes: UI Изменение интерфейса
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants