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

Add Magistrate, Dignitary Department, and Minor FTL Fixes #1536

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

CerberusWolfie
Copy link
Contributor

@CerberusWolfie CerberusWolfie commented Jan 14, 2025

Description

This adds the Magistrate to the role list of Central Command VIPs. This also splits up the Central Command VIPs into their own department named "Dignitary" for the purpose of making it clear that Central Command VIPs are above Command and should be responding to orders from Central Command.


TODO

Lots of stuff to do here, mostly just transferring Chief Justice's loadout to also work for Magistrate (most of it). I'll also list the basics I've done as well.

Magistrate-Specific

  • Add Magistrate-only Access (for locker).
  • Add Magistrate Job, Job Icon, and PlayTimeTracker
  • Add Magistrate clothes (ported from PS13)
  • Add Magistrate job spawner.
  • Add Magistrate Locker Teleporter and Locker.
  • Add Magistrate Contents to Locker.
  • Add Magistrate loadout (Chief Justice included).
  • Add Magistrate stamp and stamped icon.

Dignitary Department

  • Add Dignitary Department.
  • Put Magistrate, Blueshield Officer, and Nanotrasen Representative in Dignitary.
  • Add FTL for Dignitary.
  • Put Dignitary above Command.

Map Stuff

  • Add Magistrate job spawners to maps.
  • Add a Centcom Wintercoat to NTR and Magistrate on the Glacier map.\

Extras/Fixes

  • Add winter coats for Magistrate and NanoRep on Glacier.
  • Fix the category grouping FTL for NTR and BSO.

Media

Lobby/Customization Screenshots

Jobs Listing
image
Category Grouping
image

Items/Clothes/Entities

Stamps (Stamp shows the same in the spawn menu)
image
Magistrate Robes
image
Spawner & Magistrate Robes
image
Access List
image
PDA Crew Manifest
image
Lockers with Teleporter
image
Locker with Contents (from Teleported Locker)
image

Map Spawners

Arena
image
Asterisk
image
Core
image
Edge
image
Europa
image
Gaxstation
image
Glacier (Winter Coats for NTR and Magistrate as well.)
image
Hammurabi
image
Hive
image
Lighthouse
image
Meta
image
Pebble
image
Radstation
image
Saltern
image
Shoukou
image
Submarine
image
Tortuga
image

Misc. Minor Fixes

Chief Justice Access
image
Category Groupings (BSO AND NTR)
image
image


Changelog

🆑

  • add: Added Magistrate role.
  • add: Added Dignitary Department.
  • add: Added winter coats for NTR and Magistrate on Glacier Spawn.
  • tweak: Moved Blueshield Officer and Nanotrasen Representative to Dignitary Department.
  • fix: Fixed the category groups identifier for BSO and NTR in Loadouts.
  • fix: Fixed Chief Justice tag in ID Card Computer.

@github-actions github-actions bot added Changes: Localization Changes any ftl files Changes: YML Changes any yml files Changes: Sprite Changes any png or json in an RSI labels Jan 14, 2025
Copy link
Contributor

github-actions bot commented Jan 14, 2025

RSI Diff Bot; head commit e8a9e88 merging into 78347d7
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Objects/Misc/bureaucracy.rsi

State Old New Status
paper_stamp-magistrate Added

Resources/Textures/_Goobstation/Clothing/Uniforms/Jumpsuit/magistrate.rsi

State Old New Status
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/_Goobstation/Interface/Misc/job_icons.rsi

State Old New Status
magistrate Added

Resources/Textures/_Goobstation/Objects/Devices/unique_teleporters.rsi

State Old New Status
mag Added

Resources/Textures/_Goobstation/Objects/Misc/stamps.rsi

State Old New Status
stamp-magistrate Added

Resources/Textures/_Goobstation/Structures/Storage/closet.rsi

State Old New Status
mag Added
mag_door Added

Edit: diff updated after e8a9e88

@github-actions github-actions bot added the Changes: Map Changes any yml file in the Maps directories label Jan 14, 2025
@CerberusWolfie
Copy link
Contributor Author

Not going to update branch until the last minute to prevent potential merge conflicts with changelogs that get updated into the branch. I intend to finish this tomorrow (today if you're in ET or further).

@CerberusWolfie CerberusWolfie changed the title Add Magistrate and Make Dignitary Department Add Magistrate, Dignitary Department, and Minor FTL Fixes Jan 15, 2025
@CerberusWolfie CerberusWolfie marked this pull request as ready for review January 15, 2025 06:13
@CerberusWolfie
Copy link
Contributor Author

Me when I leave gimp file in .rsi without realizing.

@Mike32oz
Copy link
Contributor

Right now I'm working on meta I'm adding the holopads. This may override if I push the RP

@CerberusWolfie
Copy link
Contributor Author

Right now I'm working on meta I'm adding the holopads. This may override if I push the RP

Uh, depends. I don't think it would override the spawners? I'd hope not. 👍

If it does, I mean, not a hard thing to fix. All I did was add spawners.

@CerberusWolfie
Copy link
Contributor Author

On a side note, not sure why 4 tests failed, but I'm going to blame it on the tests and not my additions. :3

@sleepyyapril
Copy link
Contributor

On a side note, not sure why 4 tests failed, but I'm going to blame it on the tests and not my additions. :3

Correct!

@sleepyyapril
Copy link
Contributor

Gonna say this one needs a @VMSolidus approval

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 Status: Merge Conflict FIX YOUR PR AAAGH label Jan 15, 2025
VMSolidus pushed a commit that referenced this pull request Jan 15, 2025
<!--
This is a semi-strict format, you can add/remove sections as needed but
the order/format should be kept the same
Remove these comments before submitting
-->

# Description

<!--
Explain this PR in as much detail as applicable

Some example prompts to consider:
How might this affect the game? The codebase?
What might be some alternatives to this?
How/Who does this benefit/hurt [the game/codebase]?
-->

Adds Cargo & Lawyer access to NanoTrasen Representative
Adds Cargo access for Blueshield Officer

NTR is given lawyer to allow them to access the brig. NTR has brig
access but is unable to enter brig on most maps due to brig access not
being used, so gave them lawyer access.
NTR is also given cargo access so they can write reports about
logistics.
BSO should have basic access to all departments, logistics was missing
so added cargo.

Should hopefully not interfere with PR #1536 as it only changes BSO &
NTR

---

# Changelog

<!--
You can add an author after the `:cl:` to change the name that appears
in the changelog (ex: `:cl: Death`)
Leaving it blank will default to your GitHub display name
This includes all available types for the changelog
-->

:cl:
- tweak: Tweaked the access of NTR to include Lawyer & Cargo
- tweak: Tweaked the access of BSO to include Cargo

---------

Co-authored-by: sleepyyapril <[email protected]>
@CerberusWolfie
Copy link
Contributor Author

I'll fix the meta map when I get back. I'm in lab for another 3.5 hours, haha.

@CerberusWolfie
Copy link
Contributor Author

Merge conflicts solved, but the bot doesn't like me, so it decided not to remove them. :3

@github-actions github-actions bot removed the Status: Merge Conflict FIX YOUR PR AAAGH label Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Changes any ftl files Changes: Map Changes any yml file in the Maps directories Changes: Sprite Changes any png or json in an RSI Changes: YML Changes any yml files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants