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

Adds SolGov shuttle for players to play on #468

Draft
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

Bam40002
Copy link
Contributor

@Bam40002 Bam40002 commented Aug 27, 2023

What does it add?

Adds a playable SolGov shuttle.

Im planning to make it choose between map files so not every round the shuttle spawns in is the same map and the layout of the shuttle is different. (broken english)

Why is it good for the game?

Could lead to fun interactions with the intrepid.

Authorship

Me (based a bit on egors OBS shuttle with the code)

Changelog

🆑 Bam4000
rscadd - Added offship solgov shuttle role
/:cl:

@EgorDinamit EgorDinamit added the 🍁1 Map Anything mapping-related. label Aug 28, 2023
@Bam40002 Bam40002 force-pushed the sol-shuttle branch 2 times, most recently from 5c4a9ff to a58e097 Compare August 29, 2023 14:38
-Adds a sol shuttle

Shuttle things

-Fixes airlock
-Random names!

(Still broken)

Varying Variable Troubles

-Changed the format to TGM
-Almost fixed the Airlock

Stupid Quotes

-added a " to a thing

Mapping Fixes

-Fixed some duplicate things on top of eachother.
-Removed some atmospherics pipes going into nowhere.

Mapping fixed.

-This map has issues and I fixed a bit

Duplicate Crap

-Catwalks on catwalks removed

/obj/effect/overmap/visitable/ship/landable/sol_shuttle
name = "SolGov Cruiser"
desc = "Sensor array detects a small SolGov Cruiser."
Copy link
Member

Choose a reason for hiding this comment

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

Same here, but it could also use a bit more text to describe some details, maybe?

vessel_size = SHIP_SIZE_SMALL

/obj/effect/overmap/visitable/ship/landable/sol_shuttle/New()
name = "SEV [pick("Quantum", "Enterprise", "Event Horizon", "Notch","Discovery","Vagrant")]"
Copy link
Member

Choose a reason for hiding this comment

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

I'd imagine "SEV" doesn't exactly fit there, no? "SEV" stands for "Sol Exploratory Vessel", if I am not mistaken.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will change it to SFV meaning Sol Fleet Vessel.

name = "SEV [pick("Quantum", "Enterprise", "Event Horizon", "Notch","Discovery","Vagrant")]"
for(var/area/sol_shuttle/A)
A.name = "\improper [name] - [A.name]"
GLOB.using_map.area_purity_test_exempt_areas += A.type
Copy link
Member

Choose a reason for hiding this comment

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

Is it needed..?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Randomly picking a name? I think so.

Copy link
Member

@EgorDinamit EgorDinamit left a comment

Choose a reason for hiding this comment

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

The jobs would certainly need a rewrite or general improvements here and there.

preset_name = "SEV Quantum"

/obj/machinery/telecomms/server/map_preset/sol_shuttle
preset_name = "SEV Quantum"
Copy link
Member

Choose a reason for hiding this comment

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

Might want to give it some generic channel name instead? Let alone, ship might not be even called that when it spawns.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

On it! The SEV Quantum was the placeholder name.

-Fixed the shuttle, it works now
-Made a few mapping changes
-Added placeholder map for the second shuttle
-Changed job descriptions a bit
-Added new job the VIP (might need to remove one)
-Mapped a bit of the second shuttle
-Tweaked the first shuttle a bit
-Made it so when undocked the shuttle still has gravity
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍁1 Map Anything mapping-related.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants