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

Shuttle clean up follow up #1684

Merged

Conversation

ErhardSteinhauer
Copy link
Contributor

@ErhardSteinhauer ErhardSteinhauer commented Jul 16, 2024

About the PR

PR will be in draft until changes made to the shuttles are tested.
Following yml-edits to the files were made:

  • Removed job spawners.
  • Removed trash spawners.
  • Removed kitchen related machines (primarily microwaves) from non-food trucks.
  • Removed food.
  • Replaced filled weapon safes with empty ones.
  • Removed Chemical machines and supplies from non-medical shuttles.
  • Removed medical items from non-medical shuttles: removed medkits, filled cabinets are replaced with empty ones, vendomats replaced either with cabinets or machine frames.

Why / Balance

Clean up, push towards narrower shuttle specialization.

How to test

  1. Load shuttles from the list

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Possible everything

Changelog
🆑 erhardsteinhauer

  • remove: Removed food items and machines from non-food shuttles.
  • remove: Removed medical/chemistry related items and machines from non-medical shuttles.

@arimah
Copy link
Contributor

arimah commented Jul 16, 2024

Re Stellaris: That's a service ship with an integrated kitchen. The idea is that it can serve simpler meals and refreshments alongside whatever performance is being put on. Removing its microwave and ChefVend leaves it with a crippled, non-functional kitchen. The ship should either be reworked not to include a kitchen at all, or be allowed to keep its kitchen. If the kitchen is removed, the ship will likely see even less use as one of its selling points is negated.

@ErhardSteinhauer
Copy link
Contributor Author

Re Stellaris: That's a service ship with an integrated kitchen. The idea is that it can serve simpler meals and refreshments alongside whatever performance is being put on. Removing its microwave and ChefVend leaves it with a crippled, non-functional kitchen. The ship should either be reworked not to include a kitchen at all, or be allowed to keep its kitchen. If the kitchen is removed, the ship will likely see even less use as one of its selling points is negated.

irc the original PR, the shuttle was supposed to be a theatre. and the PR was submitted before the guidelines were a thing (I think), hence the kitchen.

@ErhardSteinhauer
Copy link
Contributor Author

Hm. Perhaps Stellaris should be delisted until someone reworks it?

@arimah
Copy link
Contributor

arimah commented Jul 16, 2024

the shuttle was supposed to be a theatre

I mean... theatres in real life also sell food. That's where the profits are!

@dvir001
Copy link
Contributor

dvir001 commented Jul 16, 2024

the shuttle was supposed to be a theatre

I mean... theatres in real life also sell food. That's where the profits are!

Alot of them do catering or outside service and don't actually hire kitchen stuff.

@whatston3
Copy link
Contributor

I mean... theatres in real life also sell food. That's where the profits are!

What if there was a specialized "junk food prep kit" you got for theatre concessions instead of ye olde stocked pantry?

Okay, you can serve food, but it's a pack of C&Ds, DIY popcorn and a large soda.

@dustylens
Copy link
Contributor

"A medium-sized shuttle, with a large public area equipped with a stage with camera and kitchen facilities"
- The Shipyard

The ship is of medium size and offers two primary services: Entertainment and kitchen. The theater portion is a fluff piece for entertainment and community engagement in addition to providing variation in settings for restaurant ships. This is not a "salvage ship" with a kitchen stuffed in the back next to a chem vend and a science room.

This is a dinner and a show ship and it offers an opportunity for players who like to specialize in both to engage in that role. It's well within even the hand wavy balance guidelines.

At the most I could see it receiving a redress to have the seating area differentiated from the movie theater experience.

@github-actions github-actions bot added the Merge Conflict This PR has conflicts that prevent merging label Jul 17, 2024
Copy link
Contributor

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

@ErhardSteinhauer
Copy link
Contributor Author

You know what? I'll redo this PR and make it a bit cleaner. And will not touch vending machines in updated version.

@github-actions github-actions bot added Map-Shuttle Map - Shuttle No C# labels Jul 17, 2024
@github-actions github-actions bot removed the Merge Conflict This PR has conflicts that prevent merging label Jul 17, 2024
@ErhardSteinhauer
Copy link
Contributor Author

Reworked the PR, should be good for testing

@ErhardSteinhauer
Copy link
Contributor Author

Left the kitchen on that theatre shuttle untouched

@ErhardSteinhauer
Copy link
Contributor Author

Right, so I bought every shuttle I changed. They spawned upon purchase without errors.

@dvir001 dvir001 merged commit a53c107 into new-frontiers-14:master Jul 18, 2024
23 checks passed
FrontierATC added a commit that referenced this pull request Jul 19, 2024
@arimah arimah mentioned this pull request Aug 3, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Map-Shuttle Map - Shuttle No C# Status: Needs Review This PR is awaiting reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants