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] Flavor Roleplaying Overhaul #2274

Merged
merged 17 commits into from
Jul 28, 2023

Conversation

Bobbanz1
Copy link
Contributor

About The Pull Request

Expands on what was ported in this PR:

By porting the Following PRs from Skyrat-TG:

Also random PR from TG which was needed for this thing:

ALSO
Introduces a new character setup tab to contain all the relevant roleplaying stuff in it, what this means is that I have moved the Flavor Text to this new place, and placed everything else that was introduced in this PR (examine panel not included) in that preference tab.

Picture of New Tab

Screenshot 2023-01-13 200533

Why It's Good For The Game

Testing Photographs and Procedure

Screenshots&Videos
2023-01-13.19-36-21.mp4

Changelog

🆑Azarak, NotRanged, FlamingLily, nikothedude, Trigg, Iamgoofball, Gandalf2k15, GoldenAlpharex
add: Added flavor text functionality for silicons.
fix: General, Medical, and Security records work again.
add: Security HUDs and medical HUDs can now view sec/medical records on examine, respectively, and can also view general records.
add: Re-implemented silicon flavor texts!
code: Vastly improved the code of the Examine menu's Character Preview, making it a lot more reflective of what the character currently looks like, while also making it much more performant at the same time!
fix: Cyborgs will now appear in the Character Preview (without their lights, sadly) of the Examine panel too!
fix: The examine panel's character preview no longer shows you instead of the person you're looking at.
fix: The Examine Panel should now properly render characters, rather than just a select few mutant bodyparts.
fix: The Examine Panel's preview will now render characters always standing upright, and at 100% size, for consistency.
refactor: Flavor Examine thingy has been replaced with the Examine Panel and all code related to the previous one has been either removed outright or trimmed down.
/:cl:

Copy link
Contributor

@Kmc2000 Kmc2000 left a comment

Choose a reason for hiding this comment

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

Since this involves DB queries, I am hesitant to touch this. Have you tested this on your local branch?

@Bobbanz1
Copy link
Contributor Author

Bobbanz1 commented Mar 13, 2023

Since this involves DB queries, I am hesitant to touch this. Have you tested this on your local branch?

I'm using a MySQL version but this is how it looks on my local branch.

Screenshot 2023-03-13 195113

covertcorvid
covertcorvid previously approved these changes Mar 24, 2023
@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@Bobbanz1
Copy link
Contributor Author

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

Don't let the door hit you on the way out

Copy link
Contributor

@Kmc2000 Kmc2000 left a comment

Choose a reason for hiding this comment

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

DB changes ahead..
@Bokkiewokkie any idea how we handle those? back in my day there was no db!

@covertcorvid
Copy link
Member

DB changes ahead.. @Bokkiewokkie any idea how we handle those? back in my day there was no db!

I know you have to load the new database schema before you run the testmerge, but I'm not sure how that's done without disrupting the current contents of the database

@github-actions
Copy link

github-actions bot commented Jul 7, 2023

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

@Bobbanz1
Copy link
Contributor Author

Bobbanz1 commented Jul 7, 2023

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

Fuck it I'll fix you later

@Bokkiewokkie
Copy link
Contributor

DB changes ahead.. @Bokkiewokkie any idea how we handle those? back in my day there was no db!

Yeah it's very easy dw

code/datums/datacore.dm Outdated Show resolved Hide resolved
code/datums/datacore.dm Outdated Show resolved Hide resolved
code/datums/datacore.dm Outdated Show resolved Hide resolved
code/datums/outfit.dm Outdated Show resolved Hide resolved
tgui/packages/tgui/components/Box.tsx Outdated Show resolved Hide resolved
tgui/packages/tgui/components/Box.tsx Outdated Show resolved Hide resolved
Copy link
Contributor

@Bokkiewokkie Bokkiewokkie left a comment

Choose a reason for hiding this comment

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

🚀

@Bokkiewokkie Bokkiewokkie merged commit 126ea59 into BeeStation:master Jul 28, 2023
9 checks passed
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.

4 participants