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

Tegu-class Communications Freighter #3016

Merged
merged 25 commits into from
Dec 7, 2024

Conversation

goober3
Copy link
Member

@goober3 goober3 commented May 22, 2024

About The Pull Request

This PR adds a brand new ship to Nanotrasen's post-rework roster.

Images

68747470733a2f2f616666656374656461726330372e626c6f622e636f72652e77696e646f77732e6e65742f6d6462322f696d616765732f3336373235323037352f33333231373933373937312f612f5f6d6170735f73687574746c65735f6e616e6f74726173656e5f

images below are outdated
image

image

The Tegu-class is an engineering freighter equipped with a full telecommunications setup and a subshuttle. It is built to not only facilitate communications between Nanotrasen-aligned vessels in the sector, but also foster several smaller networks for other vessels with a small array of relays. The ship also comes equipped with a small cargo department, an engineering department, a small communications room, and a supply office for the QM.

This PR also adds a new design disk for printing telecommunications parts.

To-do:

  • Ship description.
  • In-game testing.
  • Add security role.
  • Mining facilities.

Why It's Good For The Game

more ship

Changelog

🆑goober3, Cloudbreak
add: The year is 2070. The Tegu-class Communications Freighter has just been fullmerged.
add: A new subshuttle, the Skink-class Cargo Runner.
/🆑

@goober3 goober3 requested a review from a team as a code owner May 22, 2024 18:50
@github-actions github-actions bot added Map Change Tile placing is hard. Thank you for your service. Code change Watch something violently break. labels May 22, 2024
@FalloutFalcon FalloutFalcon added the Ship Ship successfully tested. label May 24, 2024
@EEASAS
Copy link

EEASAS commented Jun 1, 2024

okkk.... but isnt there like.... new ship restrictions.... like you cant make new nanotrasen/independent/syndicate/srm/solgov ships......

@meemofcourse
Copy link
Contributor

do they know chat

@goober3
Copy link
Member Author

goober3 commented Jun 1, 2024

okkk.... but isnt there like.... new ship restrictions.... like you cant make new nanotrasen/independent/syndicate/srm/solgov ships......

i'm doing this in defiance of the maptainers because i hate them

@thgvr
Copy link
Member

thgvr commented Jun 1, 2024

okkk.... but isnt there like.... new ship restrictions.... like you cant make new nanotrasen/independent/syndicate/srm/solgov ships......

This restriction is for contributors outside of the main team.

@goober3 goober3 marked this pull request as draft July 5, 2024 20:18
@rye-rice
Copy link
Member

sowwy, nwo prwogwess iwn mwonths, rweopwen wwen wrorking on iwt agwian

@rye-rice rye-rice closed this Jul 12, 2024
@goober3
Copy link
Member Author

goober3 commented Aug 24, 2024

we're so back

@goober3 goober3 reopened this Aug 24, 2024
@goober3 goober3 marked this pull request as ready for review August 26, 2024 14:19
@thgvr
Copy link
Member

thgvr commented Aug 27, 2024

I am begging you to finish this

@goober3
Copy link
Member Author

goober3 commented Aug 27, 2024

I am begging you to finish this

this is technically already finished. it just needs a review and a description.

a telecomms guide too maybe

Copy link
Contributor

@meemofcourse meemofcourse left a comment

Choose a reason for hiding this comment

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

imagen
i do not like that the lightswitch in the officer's lounge controls the lights in the officers' quarters. there are also three light switches in the cargo office
imagen
this restroom needs better lighting
imagen
i believe the pda painter is a centcom utility and it lets you have SUNS pdas so it would be best to remove it
imagen
the only set of tools i could find in this ship are hidden below two welding helmets. consider adding toolboxes and toolbelts to the engineer and engi director lockers - they should not start with them
imagen
maybe have an emergency suit here
imagen
while i'm not an engi player i don't think this is right

maybe a ship like this could use a power monitor. i'd ask a maptainer/other person who is more experienced with telecommunications to test things out but cool ship

@goober3
Copy link
Member Author

goober3 commented Sep 3, 2024

i do not like that the lightswitch in the officer's lounge controls the lights in the officers' quarters. there are also three light switches in the cargo office

this was done so i wouldn't have to make more areas and add APCs to the officer's quarters but in hindsight i probably should just do that. also the multiple lightswitches exist for the same reason you'd have two light switches on both ends of a hallway. it's more convenient like that.

i believe the pda painter is a centcom utility and it lets you have SUNS pdas so it would be best to remove it

i might add a generic sanitised version of the PDA painter later in a seperate PR. for now they can live without.

thak yuo

-added ship description thank you apogee
-made officer quarters unique areas
-replaced non-bridge computers with retro computers
-added power monitor
-added emergency suits
-added an extra light to the bathroom
-added toolbelts + boxes to all engineer lockers, except for an extra CE belt because the CE spawns with a set already and i don't want multiple here
-replaced the props on the supply director's desk with a book and pencil because the lamp clipped with the window
-added the electrical closet back because PDA painters allow painting of faction PDAs
@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Sep 25, 2024
Copy link
Contributor

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

@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Oct 10, 2024
@thgvr
Copy link
Member

thgvr commented Nov 3, 2024

@shiptest-ss13/maptainers ancient pr, please decide if its being merged or not

@goober3
Copy link
Member Author

goober3 commented Nov 4, 2024

@shiptest-ss13/maptainers ancient pr, please decide if its being merged or not

this still has yet to be tested substantially in-game actually. some day...

@DorfGuy1
Copy link

DorfGuy1 commented Nov 7, 2024

Where's the guide?
TO the comms

@tzui5020
Copy link

It needs a chair or a few railings in the Cargobay please. I beg you

@thgvr
Copy link
Member

thgvr commented Dec 3, 2024

purgatory

Copy link
Contributor

@meemofcourse meemofcourse left a comment

Choose a reason for hiding this comment

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

BWAAH SCARY TELECOMMS

@meemofcourse meemofcourse added this pull request to the merge queue Dec 7, 2024
Merged via the queue into shiptest-ss13:master with commit 9ccce8f Dec 7, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break. Map Change Tile placing is hard. Thank you for your service. Ship Ship successfully tested.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants