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

Drastic Changes to the USS Almayer research department #3962

Merged
merged 15 commits into from
Jul 28, 2023

Conversation

spartanbobby
Copy link
Contributor

@spartanbobby spartanbobby commented Jul 21, 2023

About the pull request

This PR changes research onboard the USS Almayer to something I hope is better to use and visually better

I'd like to see this test merged to gauge community feedback/what needs changing

Explain why it's good for the game

I believe this is an improvement to the research department making containment an actually secure area and making the research part of the department more open and closer together hopefully making it easier to move around

this PR adds alot of shutters and blastdoors to research one major blastdoor acts as a master lockdown basically shutting down the entirety of the department, no one in, no one out

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
maptweak: heavy changes to the research department
/:cl:

@github-actions
Copy link
Contributor

You currently have a negative Fix/Feature pull request delta of -45. Maintainers may close this PR at will. Fixing issues or improving the codebase will improve this score.

@github-actions github-actions bot added the Mapping did you remember to save in tgm format? label Jul 21, 2023
@Nanu308 Nanu308 added the Mapping Approved adds 500 new dict keys label Jul 22, 2023
@Nanu308
Copy link
Member

Nanu308 commented Jul 22, 2023

lgtm

Copy link
Member

@Nanu308 Nanu308 left a comment

Choose a reason for hiding this comment

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

👍

@Segrain
Copy link
Contributor

Segrain commented Jul 22, 2023

I will make a more detailed review later, but one thing makes itself outrageously evident even at a passing glance: that massive W-Y logo does not belong there.

@Segrain Segrain self-requested a review July 22, 2023 08:25
@spartanbobby
Copy link
Contributor Author

I will make a more detailed review later, but one thing makes itself outrageously evident even at a passing glance: that massive W-Y logo does not belong there.

It's there because it looks cool will remove on maptainer request

@stagda stagda mentioned this pull request Jul 28, 2023
3 tasks
@Nanu308 Nanu308 added this pull request to the merge queue Jul 28, 2023
Merged via the queue into cmss13-devs:master with commit e742d13 Jul 28, 2023
cm13-github added a commit that referenced this pull request Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mapping Approved adds 500 new dict keys Mapping did you remember to save in tgm format?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants