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

[TEMP] Disable loadout costs #230

Closed
wants to merge 1 commit into from

Conversation

AsciiSquid
Copy link
Contributor

About The Pull Request

Workaround to loadouts while testcoin economy is getting rebalanced. Disables the gear menu's ability to charge players metacoins for items. Prices will still be listed but everything is actually free for all intents and purposes.

Why It's Good For The Game

Most articles of clothing currently cost more than an entire spacecraft.

Changelog

🆑
tweak: Loadout gear is now temporarily free!
/:cl:

@keywordlabeler keywordlabeler bot added the Tweak label Sep 3, 2021
@Azarak
Copy link
Contributor

Azarak commented Sep 3, 2021

Won't this give free stuff to savefiles...

@AsciiSquid
Copy link
Contributor Author

Yes

@Azarak
Copy link
Contributor

Azarak commented Sep 3, 2021

Then it's not a temporary change, because the savefiles are gonna be affected

@AsciiSquid
Copy link
Contributor Author

It's temp in that I wouldn't recommend permanently merging this pr for that exact reason.

@shiptest-server
Copy link
Member

Test Merge Deployed

Server Instance

shiptest

Revision

Origin: 033ea7d
Pull Request: a7399a3
Server: dd05c1250d57bfccc2a44d8462a4c502f044d617

@shiptest-server
Copy link
Member

Test Merge Updated

Server Instance

shiptest

Revision

Origin: 033ea7d
Pull Request: a7399a3
Server: 7c123be6a9d9ffcf192aa21466691c922a728b54

@shiptest-server
Copy link
Member

Test Merge Updated

Server Instance

shiptest

Revision

Origin: 9693b43
Pull Request: a7399a3
Server: 651bb7bd5250da841cf57899f6ff939ae8b47202

@ZephyrTFA ZephyrTFA added the Test Merged They've done it for free. label Sep 7, 2021
@shiptest-server
Copy link
Member

Test Merge Updated

Server Instance

shiptest

Revision

Origin: ca83a63
Pull Request: a7399a3
Server: b9679cc28f47ef4054511559ac3e5f35dd3100bb

@shiptest-server
Copy link
Member

Test Merge Updated

Server Instance

shiptest

Revision

Origin: ca83a63
Pull Request: a7399a3
Server: 52f18b307da9ed9bafb2f173c5db9806fde13a70

@MarkSuckerberg MarkSuckerberg added the Do Not Merge You probably made someone angry, or you're stopping people from getting angry. label Sep 19, 2021
@ZephyrTFA ZephyrTFA marked this pull request as draft November 20, 2021 18:02
@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2022

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@MarkSuckerberg MarkSuckerberg marked this pull request as ready for review February 6, 2022 07:10
@ZephyrTFA ZephyrTFA removed the Test Merged They've done it for free. label Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge You probably made someone angry, or you're stopping people from getting angry. Merge Conflict Use Git Hooks, you're welcome. Tweak
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants