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

MailPod (New) #1197

Merged
merged 15 commits into from
Apr 29, 2024
Merged

MailPod (New) #1197

merged 15 commits into from
Apr 29, 2024

Conversation

dvir001
Copy link
Contributor

@dvir001 dvir001 commented Apr 5, 2024

About the PR

Added to new Mail Pod ship for the mail man

Why / Balance

Small, effective and reliable mail ship for mail operations.
Build to be able to dock to all ships, with intention to support a bigger mail ship in the future.

Media

image

Breaking changes

N/A

Changelog
🆑 dvir01

  • add: Added the new mail ship, the Mail Pod, available for the mail carrier in the mail office.

@dvir001 dvir001 requested a review from MagnusCrowe April 5, 2024 11:19
@github-actions github-actions bot added S: Needs Review This PR is awaiting reviews Map-Shuttle Map - Shuttle No C# labels Apr 5, 2024
@Wolfking6116
Copy link
Contributor

Wolfking6116 commented Apr 5, 2024

I thought there was a new rule that APU weren't allowed to be mapped on ships anymore, outside of Sec?

@ErhardSteinhauer
Copy link
Contributor

uh-oh! someone got caught smuggelin' dem APUs in his ship!

@ThatOneGoblin25
Copy link
Contributor

We need wall mounted Pacman variants fr

@dvir001
Copy link
Contributor Author

dvir001 commented Apr 5, 2024

I thought there was a new rule that APU weren't allowed to be mapped on ships anymore, outside of Sec?

Its a mail man only ship.

@Wolfking6116
Copy link
Contributor

What was the purpose of nerfing the original then...? 😅

@dvir001
Copy link
Contributor Author

dvir001 commented Apr 8, 2024

What was the purpose of nerfing the original then...? 😅

Original one used to be public facing.
After rework truck will also be getting them.

Copy link
Contributor

github-actions bot commented Apr 8, 2024

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

@github-actions github-actions bot added the S: Merge Conflict This PR has conflicts that prevent merging label Apr 8, 2024
@MoistBiscuits
Copy link
Contributor

Im not sure why this ship has an ATM, mail carriers are on SR's payroll.

@dvir001
Copy link
Contributor Author

dvir001 commented Apr 8, 2024

Im not sure why this ship has an ATM, mail carriers are on SR's payroll.

Tipping

@github-actions github-actions bot removed the S: Merge Conflict This PR has conflicts that prevent merging label Apr 9, 2024
@dvir001 dvir001 added S: Awaiting Changes This PR has changes that need to be made before merging and removed S: Needs Review This PR is awaiting reviews labels Apr 18, 2024
Copy link
Contributor

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

@github-actions github-actions bot added the S: Merge Conflict This PR has conflicts that prevent merging label Apr 19, 2024
@github-actions github-actions bot removed the S: Merge Conflict This PR has conflicts that prevent merging label Apr 26, 2024
@dvir001
Copy link
Contributor Author

dvir001 commented Apr 26, 2024

Updated.

@dvir001 dvir001 added S: Needs Review This PR is awaiting reviews and removed S: Awaiting Changes This PR has changes that need to be made before merging labels Apr 26, 2024
@github-actions github-actions bot added the S: Merge Conflict This PR has conflicts that prevent merging label Apr 26, 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 S: Merge Conflict This PR has conflicts that prevent merging label Apr 26, 2024
@MagnusCrowe MagnusCrowe merged commit c3c5b7f into new-frontiers-14:master Apr 29, 2024
12 checks passed
FrontierATC added a commit that referenced this pull request Apr 29, 2024
@Danchi299
Copy link

Substations' input is HV Cable and output is MV Cable, it does not need a terminal, due to that the ship's power does not work.

@dvir001
Copy link
Contributor Author

dvir001 commented Apr 30, 2024

Substations' input is HV Cable and output is MV Cable, it does not need a terminal, due to that the ship's power does not work.

I'll fix it in a new pr.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Map-Shuttle Map - Shuttle No C# S: Needs Review This PR is awaiting reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants