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

Traitor flavor (take 3) uhh... #30198

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

Conversation

TokenStyle
Copy link
Contributor

@TokenStyle TokenStyle commented Jul 20, 2024

About the PR

reopen/revive #28983 PR that implements flavor, description, and some other stuff, would follow design docs.
New docs PR 264 .
Old docs PR 241 .

Why / Balance

more text, more lore, more rp.

Technical details

Changed smartly some stuff. TraitorRuleComponent.cs is ObjectiveIssuers now work as intermediate part for locales.
Added factions to ObjectivesTextGetInfoEvent. This will be useful if blood brothers ever get added and more lore, and though cuz in ss13 syndies corps are not actually a full-fledged “union”.
And please don't look in ObjectivesSystem, the summaries are god awful.

Media

screen main feat3

339755331-53e9c9df-7fc2-44be-a31b-d6f1c584b4ac

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

Breaking changes

Antagonists change. Though ONLY role, faction, lore descriptions and changed or added, no "fancy" discounts/coupons, antag refactor or smthin'.

Changelog

🆑

  • add: Now traitors have more than 10 variations.

@github-actions github-actions bot added S: Needs Review Status: Requires additional reviews before being fully accepted S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted labels Jul 20, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Catdere
Copy link

Catdere commented Jul 20, 2024

This is something Ive been wanting for a while now, As someone who enjoys acting as a member of the given corp its always a weird moment asking another traitor who they work for. This is a huge RP QOL feature and better fleshes out the Spinward Syndicate corporations and groups.

@Futuristic-OK
Copy link
Contributor

This also good improvement for MRP servers. LRP is RP too!

@TokenStyle
Copy link
Contributor Author

Wait for design document review PR 264

@ElPidor

This comment was marked as spam.

@ElPidor

This comment was marked as spam.

@whateverusername0
Copy link

whateverusername0 commented Jul 25, 2024

wew lad
keep it up 💪

@Boaz1111
Copy link
Contributor

Boaz1111 commented Sep 9, 2024

CL isn't informative enough most likely

@TokenStyle
Copy link
Contributor Author

CL isn't informative enough most likely

This is draft, most info in design document space-wizards/docs#264

@SlamBamActionman SlamBamActionman added the S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. label Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted S: Needs Review Status: Requires additional reviews before being fully accepted S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants