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

Show contractors Employing Company+Allow alt titles #6980

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MarsM0nd
Copy link
Contributor

About The Pull Request

This fixes that contractor IDs (The faction not being NT ones, not the Traveler title) do show which other company they are employed by. At the same time, it allows them to use alternative titles for the jobs.

Why It's Good For The Game

Showing which company someone belongs to on the ID was intended, and did just break. Alternative titles were explicitly disabled, due to confusing titles coming with some factions, but as there no longer are any, we can allow the full range of known station alternatives. A Vey-Med nurse probably should stay a nurse, and not become a full doctor when working at an NT station, at least title wise, as example.

Changelog

🆑
tweak: Allows contractors to have alt-titles
fix: Contractor IDs show the employing company
/:cl:

@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jan 27, 2025
@TheLordME
Copy link
Collaborator

Contractor dont get Alt-titles because certain lore contributors gave their company specific alt-titles nobody understands. That is why Alt titles disapear when you are a contractor

silicons pushed a commit that referenced this pull request Feb 2, 2025
silicons pushed a commit that referenced this pull request Feb 2, 2025
silicons pushed a commit that referenced this pull request Feb 2, 2025
@MarsM0nd
Copy link
Contributor Author

MarsM0nd commented Feb 3, 2025

#6531 did remove all of the alternative titles (which were part of necropolis and the naramid guilds), so currently, it's not required. I didn't fully remove it, just comment out the parts to indicate that it should stick to station default names.
As alternatives, either a list needs to be made for the default titles/no-default titles, to assign the rank instead, or a full refactor so the alt_title prototype gets handed there, being able to check if the background is set. The background handling could also be removed in code\modules\jobs\alt_title.dm and the hooks for it from code\modules\jobs\job.dm, to make it more likely such won't get added in the future.

@TheLordME
Copy link
Collaborator

ah, carry on then

silicons pushed a commit that referenced this pull request Feb 4, 2025
silicons pushed a commit that referenced this pull request Feb 10, 2025
silicons pushed a commit that referenced this pull request Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants