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

The Whippet, a Bare-Bones Cargo Hauler #784

Closed
wants to merge 2 commits into from

Conversation

Vytian
Copy link

@Vytian Vytian commented Dec 29, 2023

About the PR

Sometimes you want a smaller operation while still being able to haul cargo and haul it fast. Does the loader have too many luxuries that could be taken up by cargo space? This fuselage with thrusters attached is sure to get you and your cargo to its destination nice and quick, with minimal space dedicated to crew amenities to maximize cargo space. Buy a Whippet today!

Why / Balance

It'd be nice to have some more variety from just seeing the Loader whenever crews of one or two want to run cargo. Costs 17500 spesos to be competitive with the Loader.

Technical details

Simply adds the Whippet shuttle model to the shipyard.

Media

Whippet

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

Changelog

  • add: NC Whippet ship model to the shipyard

@github-actions github-actions bot added Map-Shuttle Map - Shuttle Map labels Dec 29, 2023
Copy link
Contributor

@FoxxoTrystan FoxxoTrystan left a comment

Choose a reason for hiding this comment

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

• Missing LateJoin Spawn.
• Emercency O2/Captain locked need to use the Filled version and not fill it yourself.


- type: gameMap
id: Whippet
mapName: 'NC Whippet'
Copy link
Contributor

@FoxxoTrystan FoxxoTrystan Dec 30, 2023

Choose a reason for hiding this comment

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

• Change mapname to "Whippet" and not "MC Whippet" to avoid long names in IFF.

I also recomend ID to have no caps in the names.

Copy link
Contributor

@TsjipTsjip TsjipTsjip left a comment

Choose a reason for hiding this comment

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

See FoxxyTrystan's notes, additionally I would like to request you look into how your shuttle looks without all those catwalks everywhere in the cargo area. In my opinion, these make your shuttle look very busy.

Did not test this shuttle in-game, but the files look fine aside from already made notes.

- Cleaned up excess decals
- Named docking doors
- Swapped service doors to bridge with command doors
- Modified names in files to remain in line with shipyard naming conventions without cluttering IFF names
@Vytian
Copy link
Author

Vytian commented Jan 1, 2024

Updated with the recommended changes. LateJoin Spawnpoint was already present, just under the cargotechnician spawnpoint instead of over it for some reason.

@dvir001 dvir001 removed the Map label Jan 2, 2024
@Everturning
Copy link

gonna say it. it looks like a....

inappropriate thing.

@Everturning
Copy link

move the engines to the center more
add some sort of a drink

@MagnusCrowe MagnusCrowe self-requested a review January 4, 2024 01:02
@github-actions github-actions bot added the Status: Needs Review This PR is awaiting reviews label Jan 4, 2024
@MagnusCrowe
Copy link
Contributor

Going to close this for now due to inactivity. Please open it up again when you come back and work on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Map-Shuttle Map - Shuttle Status: Needs Review This PR is awaiting reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants