-
Notifications
You must be signed in to change notification settings - Fork 628
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
Added an NFSD prisoner fill with headsets #2850
base: master
Are you sure you want to change the base?
Conversation
Do not use StorageFill, use the new system. do not give them |
Could you point me where to look at the replacement for StorageFill? |
Look at upstream locker fills |
Took a look at the lockers. They look good and the fill seems nice. This seems like a pretty great addition to address the need to dress prisoners up in appropriate attire without stepping into the realm of bloat or questions regarding channels and other pieces of baggage that might come with the delta headsets. There may be some parts of dvir's feedback that I'm missing. I'm not sure what some of the issues raised are. This seems like a pretty good stopgap between what we have now and a much larger PR of what might be ideal. Adding new prisoner channels/etc is way outside of the scope of this PR? While it's not my business I would have enjoyed seeing the locker and the mapping in different PRs but, you know. I'm just some guy. |
Hello! I thought I'd lend a hand here; I've got a set of suggestions that I think would make sense. All of the changes are up here, look it over if you can - feel free to take from it, get me to merge it, or comment as you like: https://github.com/Jakumba/frontier-station-14/compare/nfsd-prisoner-fills...whatston3:frontier-station-14:2025-02-10-prisoner-fill-suggestions?expand=1 In summary:
Seems reasonable to me, happy to discuss if you disagree. |
About the PR
Why / Balance
Space Law requires that prisoners be given access to the Common radio channel. This is actually quite difficult to do unless the NFSD ship commander has pre-purchased headsets. This removes that onus and ensures that prisoners will always have a headset available.
Because this touches the Hypno's I'd appreciate @MagnusCrowe's eye over it, to make sure that the map changes are acceptable.
How to test
Media
N/A
Requirements
Breaking changes
Changelog