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

L'évacuation - 1 #5052

Closed
wants to merge 74 commits into from
Closed

L'évacuation - 1 #5052

wants to merge 74 commits into from

Conversation

LC4492
Copy link
Contributor

@LC4492 LC4492 commented Nov 27, 2023

About the pull request

This PR is focused on making the evacuation pods stay stationed in the outer-hull of the Ship, and not inside it in weird rooms. With that, more internal space will be available for other mapping projects and will remove the weirdness of the original thing, making it certainly look and feel better. Its supposed to also help on where the DS would land in a hijack, giving more landing opportunities. (The CL pod is left untouched for code reasons)

Explain why it's good for the game

I don't know, you tell me.

Testing Photographs and Procedure

Upper Port mid pods
image
Upper port fore pods
image
Upper starboard fore pods
image
Upper starboard mid pods
image
Lower starboard mid pods
image
Lower port mid pods
image

Changelog

🆑
mapadd: Adds new outer-hull docks for pods. Ten pods in the upper deck, six pods in the lower deck.
maptweak: Changes the halls surrounding the pod areas and the now maintenance tunnels that were once the original pod positions, any other changes have been limited to the maximum.
/:cl:

LC4492 added 30 commits April 1, 2023 14:25
@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Nov 29, 2023
@cm13-github
Copy link
Contributor

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

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Nov 30, 2023
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Dec 2, 2023
@cm13-github
Copy link
Contributor

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

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Dec 2, 2023
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Dec 7, 2023
@cm13-github
Copy link
Contributor

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

@LC4492 LC4492 closed this Dec 9, 2023
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? Merge Conflict PR can't be merged because it touched too much code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants