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

Senior ID cards get a custom job name: Attempt 3 #1425

Merged
merged 28 commits into from
Aug 6, 2024

Conversation

WarMechanic
Copy link
Contributor

@WarMechanic WarMechanic commented Jun 22, 2024

About the PR

Taking the Senior ID card for jobs that support it will change your job title and icon to match that of the senior role.

My advice to heads of staff: Treat your seniors like they have 60 hours of playtime. Delegate them to teach recruits properly, do things that are hard and condemn them harshly for incompetence.

Why / Balance

Fixes #1263

Technical details

This solution reverts #1218 and moves most of the job name changing logic to take place in C.S/.../GameTicker.Spawning to catch when the player spawns both at round-start and the middle of the round. Various overrides in a few scripts check for VirtualJobName/Icon in a JobComponent.
No longer uses job prototypes like in Attempt 2.

Media

https://youtu.be/_YaD1ESBiwA

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

Breaking changes

Changelog

🆑

  • fix: Fixed senior job titles not displaying properly

@github-actions github-actions bot added Changes: YML Changes any yml files Changes: C# Changes any cs files labels Jun 22, 2024
@WarMechanic WarMechanic marked this pull request as ready for review June 22, 2024 05:15
Copy link
Contributor

@NullWanderer NullWanderer left a comment

Choose a reason for hiding this comment

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

I know I didn't either, but these should really be using localized job names. Looks good besides that

@github-actions github-actions bot added Changes: Documentation Changes any xml or md files Changes: Sprite Changes any png or json in an rsi labels Jul 25, 2024
Copy link
Contributor

RSI Diff Bot; head commit 709d57d merging into 09d897e
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/DeltaV/Decals/bricktile.rsi

State Old New Status
dark_box Added
dark_corner_ne Added
dark_corner_nw Added
dark_corner_se Added
dark_corner_sw Added
dark_end_e Added
dark_end_n Added
dark_end_s Added
dark_end_w Added
dark_inner_ne Added
dark_inner_nw Added
dark_inner_se Added
dark_inner_sw Added
dark_line_e Added
dark_line_n Added
dark_line_s Added
dark_line_w Added
steel_box Added
steel_corner_ne Added
steel_corner_nw Added
steel_corner_se Added
steel_corner_sw Added
steel_end_e Added
steel_end_n Added
steel_end_s Added
steel_end_w Added
steel_inner_ne Added
steel_inner_nw Added
steel_inner_se Added
steel_inner_sw Added
steel_line_e Added
steel_line_n Added
steel_line_s Added
steel_line_w Added
white_box Added
white_corner_ne Added
white_corner_nw Added
white_corner_se Added
white_corner_sw Added
white_end_e Added
white_end_n Added
white_end_s Added
white_end_w Added

@WarMechanic
Copy link
Contributor Author

i think i bunged something up, i had to pull my git files while working here in america.

@WarMechanic
Copy link
Contributor Author

all the errors have been fixed, and i have playtested the solution and found everything to be working. i think its good to go

@NullWanderer NullWanderer merged commit 10b7e40 into DeltaV-Station:master Aug 6, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Documentation Changes any xml or md files Changes: Localization Changes any ftl files Changes: Map Changes: Sprite Changes any png or json in an rsi Changes: UI Changes: YML Changes any yml files Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mission Senior ID unsuccessful
2 participants