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.

Sign up for GitHub

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

Jump to bottom

Pathfinder (Update) #926

Merged
merged 6 commits into from
Jan 30, 2024
Merged

Pathfinder (Update) #926

merged 6 commits into from
Jan 30, 2024

Conversation

ErhardSteinhauer
Copy link
Contributor

@ErhardSteinhauer ErhardSteinhauer commented Jan 26, 2024 •
edited
Loading

About the PR

Updating Pathfinder to current mapping standards:

  • Wiring rework (pulled wires from under the walls).
  • Atmos rework (simplified waste loop).
  • Replaced consoles with tabletop variants.
  • Fiddled with lockers: split salvage specialist's locker in two, replaced captain's locker with QM's locker and suit storage, replaced emergency EVA with EVA.
  • Replaced floor tiles under the diagonal walls in space with lattices.
  • Added Vacuum Markers to exposed to space tiles.
  • Enabled Pilot Job.
  • Mapped emergency lights, fire extinguisher and defibrillator cabinets.
  • Deleted about half of light source.
  • Moved ship.yml to Maps/_NF/Shuttles/
  • Updated mapchecker whitelist.

Why / Balance

New mapping standards.

Media

2024-1-28_14 20 48

  • I have added screenshots/videos to this PR showcasing its changes ingame.

Changelog
🆑 erhardsteinhauer

  • tweak: Updated Pathfinder to current mapping standards.

@github-actions github-actions bot added Map-Shuttle Map - Shuttle No C# labels Jan 26, 2024
@ErhardSteinhauer
Copy link
Contributor Author

Ship owner: @Kesiath
(Just tagging you to make sure you're aware there's suggested changes for you ship.)

@TsjipTsjip TsjipTsjip mentioned this pull request Jan 27, 2024
Open
69 tasks
@github-actions github-actions bot added the Status: Needs Review This PR is awaiting reviews label Jan 27, 2024
@github-actions github-actions bot added the Merge Conflict This PR has conflicts that prevent merging label Jan 29, 2024
@github-actions GitHub Actions
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 This PR has conflicts that prevent merging label Jan 29, 2024
@dvir001 dvir001 self-requested a review January 30, 2024 15:29
@dvir001 dvir001 merged commit c4cc5a7 into new-frontiers-14:master Jan 30, 2024
10 of 11 checks passed
FrontierATC added a commit that referenced this pull request Jan 30, 2024
@FrontierATC
Automatic Changelog (#926)
32504b7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dvir001 dvir001 dvir001 approved these changes

@MagnusCrowe MagnusCrowe Awaiting requested review from MagnusCrowe

Assignees
No one assigned
Labels
Map-Shuttle Map - Shuttle No C# Status: Needs Review This PR is awaiting reviews
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ErhardSteinhauer @dvir001