Skip to content

Cargo Console can consume caps + Communications Console adjustments#73

Open
JustLoveBeingAnOwl wants to merge 16 commits intothegrovellingpeasant:mainfrom
JustLoveBeingAnOwl:quick-red-alert-thingy
Open

Cargo Console can consume caps + Communications Console adjustments#73
JustLoveBeingAnOwl wants to merge 16 commits intothegrovellingpeasant:mainfrom
JustLoveBeingAnOwl:quick-red-alert-thingy

Conversation

@JustLoveBeingAnOwl
Copy link
Contributor

About The Pull Request

some adjustments to the cargo console and communications console
cargo can have money shoved in to adjust credit value
communications console needed fixes for red alert because that shit isn't working for whatever reason

Why It's Good For The Game

Pre-Merge Checklist

  • You tested this on a local server.
  • This code did not runtime during testing.
  • You documented all of your changes.

Changelog

🆑
add: Added new things
add: Added more things
del: Removed old things
tweak: tweaked a few things
balance: rebalanced something
fix: fixed a few things
soundadd: added a new sound thingy
sounddel: removed an old sound thingy
imageadd: added some icons and images
imagedel: deleted some icons and images
spellcheck: fixed a few typos
code: changed some code
refactor: refactored some code
config: changed some config setting
admin: messed with admin stuff
server: something server ops should know
/:cl:

Co-authored-by: John Willard <53777086+JohnFulpWillard@users.noreply.github.com>
@JustLoveBeingAnOwl JustLoveBeingAnOwl changed the title some crap Cargo Console can consume caps + Communications Console adjustments Dec 11, 2024
Copy link
Contributor

@JohnFulpWillard JohnFulpWillard left a comment

Choose a reason for hiding this comment

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

Idk how I didn't see this before. The credits thing can also go above parent call and early return instead, so like

if(credits)
do thing
return
return ..()

So you don't physically wack the console with the credits too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants