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

[Port] TGUI Manifests #11126

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

XeonMations
Copy link
Contributor

@XeonMations XeonMations commented Jun 27, 2024

About The Pull Request

TGUI-ifies the security, medical and general manifests.

Requires:

Ports:

Fixes #11095

Why It's Good For The Game

alot of newer code + better performing and nicer looking security, general and medical manifests

Testing Photographs and Procedure

Screenshots&Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
refactor: Refactored the security, medical and general manifests to TGUI.
/:cl:

@MarkusLarsson421
Copy link
Contributor

"day 1" of ??? years. :P

@XeonMations
Copy link
Contributor Author

I am holding off on this one until we can get the proper dependency PR's merged first.

I know it doesnt work shhhh
/// If the player with this job can have quirks assigned to him or not. Relevant for new player joinable jobs and roundstart antags.
#define JOB_ASSIGN_QUIRKS (1<<7)
/// Whether this job can be an intern.
#define JOB_CAN_BE_INTERN (1<<8)
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't have /tg/'s intern system

Copy link
Contributor Author

@XeonMations XeonMations Jun 28, 2024

Choose a reason for hiding this comment

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

Not yet.

Copy link
Contributor

Choose a reason for hiding this comment

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

Blows you up with my mind

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Security Record UX
3 participants