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

Base Cargo's Buy Stuff Now #17

Closed
wants to merge 2 commits into from
Closed

Conversation

Ossa88
Copy link
Collaborator

@Ossa88 Ossa88 commented Oct 6, 2024

About The Pull Request

Cargo selling pads added to all station hubs for convince of crew.

Requested by Psycho
NT Ice Base:

Indi Space Base:

NT Asteroid Base (Admin only spawn ATM)

Why It's Good For The Game

It helps the crews of various ships get rid of unwanted materials or items. Ya know a way more productive means than just leaving them on the nearest beach planet... and ya know... letting that planet take all those micro plasma... PSYCHO... THAT'S HOW YOU GET SPACE CARPS!

Changelog

🆑
add: NT Asteroid Base - Added Cargo Seller and Terminal
add: Indi Space Base - Added Cargo Seller and Terminal
add: NT Ice Base - Added Cargo Seller and Terminal
/:cl:

@Constellado Constellado added bug Something isn't working DO NOT MERGE Oh. I see. labels Oct 6, 2024
@Ossa88
Copy link
Collaborator Author

Ossa88 commented Oct 7, 2024

Alright... Going to close this.

The original request was to have a public selling terminal and pad at outposts. So did some quick mapping tossed things in and found that... Seems the code DEMANDS some interesting things.

1 - The terminal has to be on a ship
2 - The pad is unable to be removed

So to make a public terminal possible more work would need to be done and code needs to be changed... so closing... can be reopened with a proper version later

@Ossa88 Ossa88 closed this Oct 7, 2024
@Ossa88 Ossa88 deleted the Buy-my-crap branch October 7, 2024 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working DO NOT MERGE Oh. I see. Map Change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants