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

Anchorpoint ERT Station #7238

Closed
wants to merge 7 commits into from
Closed

Conversation

Steelpoint
Copy link
Contributor

@Steelpoint Steelpoint commented Sep 29, 2024

About the pull request

Adds a ERT station for CMB aligned ERT's to return to, or for admin events. The station contains a CMB fax. Anchorpoint is designed to appear larger than it actually is.

In-game image in screenshot section.

Explain why it's good for the game

Gives more love to the CMB so they have somewhere official to land. Can also be used for setting up ERTs for events or for holding prisioners.

Also has a CMB fax for roleplay oppertunities.

Testing Photographs and Procedure

Screenshots & Videos

anchorpoint

Changelog

🆑
mapadd: Anchorpoint has been added as a CMB ERT station. CMB aligned ERT's will return to this station when leaving the USS Almayer. The station also contained a CMB fax machine.
/:cl:

@cmss13-ci cmss13-ci bot added the Mapping did you remember to save in tgm format? label Sep 29, 2024
@Steelpoint
Copy link
Contributor Author

I was going to have the station contain three landing pads, so multiple ships can land on the same ERT station. However, the game will only ever use 1 of the landing pads when someone leaves on a CMB shuttle, meaning multiple Anchorpoints will be spawned for every CMB shuttle that leave.

So I've only made one of the landing pads work.

Copy link
Member

Choose a reason for hiding this comment

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

Requires more work, not up to standard compared to the other ERT stations.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In what way? More rooms? More details? Bigger? Smaller? In comparison to some of the ERT stations this map meets similar standards of size, contents, detailing and room.

@Nanu308 Nanu308 marked this pull request as draft September 29, 2024 21:20
@Steelpoint
Copy link
Contributor Author

I've gone ahead and made some cosmetic changes. I'm not really sure exactly what is not "up to standard" so I'll either leave this as it stands or wait for further direction.

@Steelpoint Steelpoint marked this pull request as ready for review September 30, 2024 08:59
@Steelpoint Steelpoint marked this pull request as draft October 2, 2024 10:57
@LC4492
Copy link
Contributor

LC4492 commented Oct 13, 2024

I suppose you should make a more complex containment site, maybe even with a smaller-sized court. It also should look more CMB-oriented, try looking at Hybrisa's CMB Local Department for inspiration.

@cmss13-ci
Copy link
Contributor

cmss13-ci bot commented Oct 27, 2024

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@cmss13-ci cmss13-ci bot added the Stale beg a maintainer to review your PR label Oct 27, 2024
@Steelpoint Steelpoint closed this Oct 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mapping did you remember to save in tgm format? Stale beg a maintainer to review your PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants