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

NFSD Outpost (New) - Split NFSD away from Frontier Outpost #840

Merged
merged 30 commits into from
Jan 27, 2024

Conversation

TsjipTsjip
Copy link
Contributor

@TsjipTsjip TsjipTsjip commented Jan 7, 2024

About the PR

Adds the NFSD outpost, a new POI which serves as the base of operations for the NFSD.

Downsizes NFSD on Frontier Outpost, replacing it with:

  • Station guard holding cell
  • Frontier staff break room (service locked)
  • Dedicated stage for the musician
  • More maints

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Initmapped:
image

Mapping view:
image

NFSD downsizing on frontier outpost:
image

Bridge updates with new vault, also janitor locker:
image

Simplified waste line:
image

Breaking changes

Changelog

🆑

  • add: Added the NFSD outpost, a new base of operations for the Sheriff and their subordinates!

@TsjipTsjip TsjipTsjip marked this pull request as draft January 7, 2024 18:36
@github-actions github-actions bot added Map-POI Map - POI C# labels Jan 7, 2024
@github-actions github-actions bot added the Status: Needs Review This PR is awaiting reviews label Jan 7, 2024
@github-actions github-actions bot added the Map-Shuttle Map - Shuttle label Jan 10, 2024
@TsjipTsjip
Copy link
Contributor Author

Okay so this is where we are at right now:

  • Spawning works
  • Station is nearly done (Needs decalling and docks need a little expanding)

This is what we still need:

  • Split bank account
  • OR a joint bank account with frontier
  • Security cargo request computer
  • Security cargo teleporter
  • Security cargo sales console
  • Station records are not station recordsering. Can't open latejoin slots.

@Cheackraze
Copy link
Member

So theres a specific bugs with station records console in that, if there are 0 records for the station, it doesnt show the late join sliders. This is a result of the late join menu portion being a child and therefore inside of the actual technical 'records' entries, but the whole UI itself just doesnt render that bit of UI if there are no records to begin with.

It could be fixed, or you could give it a quick test by spawning in to an automatic late join opened slot.

@TsjipTsjip TsjipTsjip marked this pull request as ready for review January 11, 2024 12:33
@TsjipTsjip
Copy link
Contributor Author

TsjipTsjip commented Jan 11, 2024

Ready for reviiieeewwwww!!!

This is what we have:

  • Spawning works
  • Station is in a final touches state

This is what we still need:

  • Split bank account (Future PR)
  • OR a joint bank account with frontier (Future PR)
  • Security cargo things (Future PR)
  • Downsize NFSD on Frontier Outpost (Future PR)
  • Bus (Future PR)

Potential issues:

  • If there is no sheriff, no NFSD can be hired through opening slots for them due to the above station records bug. (Minor, regular in-character hiring is still possible.)

@MagnusCrowe
Copy link
Contributor

No meteor shielding grilles?

@TsjipTsjip
Copy link
Contributor Author

No meteor shielding grilles?

Is this necessary? Frontier Outpost does not have these either, neither do most of the POI's.

@github-actions github-actions bot added the Map-Outpost Map - Outpost label Jan 17, 2024
@TsjipTsjip
Copy link
Contributor Author

We should be up-to-date with master on this branch again. NFSD on frontier outpost has been downsized already in this PR, along with wrapping up a lot more QoL things that needed to be done on Frontier. (Cargo conveyors come to mind.)

@Cheackraze
Copy link
Member

No meteor shielding grilles?

Is this necessary? Frontier Outpost does not have these either, neither do most of the POI's.

Most of the POI have all surfaces reserved for docking space is why. I tried my best to add asteroid catchers to frontier where possible but sometimes it's not. Think if it this way too, it's an opportunity to electrify them with lv/mv for an added layer of security. I would say overall it looks good, and we will likely need to just merge and plates it to figure out what does and doesn't work on it.

I will want to load this up in game and give it a proper test. I have a kind of busy weekend ahead of me but I'm aiming for Monday at the very latest to have this officially and fully reviewed.

@Cheackraze
Copy link
Member

I would say asteroid catchers aren't really necessary, however it would definitely make it look and feel better

@TsjipTsjip
Copy link
Contributor Author

I will look into asteroid shielding solutions for this POI, as I do indeed have significant non-docking surface area. Thanks for the feedback, and take your time. Willing to actively maintain this POI to bring it up to scratch during play testing.

@TsjipTsjip
Copy link
Contributor Author

Asteroid shields are in place and should be (relatively) random every round via the random spawners. I tried my best to make it look "janky", and I do believe this looks better on mass scanners as well.

@TsjipTsjip
Copy link
Contributor Author

Together with @dvir001 we found out how to anchor all POI's in space so they can't be moved. (Based on #40 )

This PR, while technically out-of-scope, anchors all POI's and Frontier outpost to a standard 99999 both linear and angular damping.

Copy link
Member

@Cheackraze Cheackraze left a comment

Choose a reason for hiding this comment

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

maps themselves look fantastic, im excited to see how this plays, just a few very minor tweaks and its good to go

Resources/Prototypes/_NF/Maps/POI/nfsd.yml Show resolved Hide resolved
- type: entity
id: SecurityFrontierOutpost
parent:
- StandardFrontierOutpost # Parent off of above
Copy link
Member

Choose a reason for hiding this comment

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

I think we need to not parent off from outposts, and instead just make this its own base class. Reason being, we need to make sure that Borg laws dont allow for some silly 'rules' while aboard an official NT outpost. Aside from that, it would only inherit 'cargo' as a component, and we are likely going to modify this some more later on to get seperate cargo accounts working.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure if this is fully addressed in 12e943e .

@github-actions github-actions bot added the Merge Conflict This PR has conflicts that prevent merging label Jan 24, 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 Merge Conflict This PR has conflicts that prevent merging label Jan 25, 2024
@dvir001 dvir001 changed the title Split NFSD away from Frontier Outpost NFSD Outpost (New) - Split NFSD away from Frontier Outpost Jan 25, 2024
@Cheackraze Cheackraze merged commit 49c4969 into new-frontiers-14:master Jan 27, 2024
10 of 11 checks passed
FrontierATC added a commit that referenced this pull request Jan 27, 2024
@TsjipTsjip TsjipTsjip deleted the nfsd-station branch January 30, 2024 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C# Map-Outpost Map - Outpost Map-POI Map - POI Map-Shuttle Map - Shuttle Status: Needs Review This PR is awaiting reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants