-
-
Notifications
You must be signed in to change notification settings - Fork 165
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
Makes the crew manifest into TGUI + Sorts the late-join menu #2662
Makes the crew manifest into TGUI + Sorts the late-join menu #2662
Conversation
Co-authored-by: celotajstg <[email protected]>
* Resolve open position issues in crew manifest * Use the classes helper for multiple classes Co-authored-by: celotajstg <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This I am adamant on
Will read this tonight or tmrw |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would like a TM, but this is good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice and simple PR. It has been testmerged for a while, so It's good to go.
About The Pull Request
Ports the following from TG:
Beeg PR, huge sanity hit.
This PR turns the manifest into TGUI, alongside sorting job stuff into better sections like dis:
Why It's Good For The Game
It looks much better and more informative
Changelog
🆑 Gboster, Celotajs, ghost
Gboster: tweak: the late-join menu should now have better seperation
Celotajs: refactor: The crew manifest has now been converted to TGUI for lobby, ghost, AI and pAI players.
Celotais: fix: Prevented departments from having negative or undefined open positions in the crew manifest
/:cl: