Skip to content

Fix joaat mixup for RemoveCurrentRise#732

Open
chipjpg wants to merge 2 commits into
citizenfx:masterfrom
chipjpg:patch-70
Open

Fix joaat mixup for RemoveCurrentRise#732
chipjpg wants to merge 2 commits into
citizenfx:masterfrom
chipjpg:patch-70

Conversation

@chipjpg

@chipjpg chipjpg commented Apr 28, 2022

Copy link
Copy Markdown
Contributor

Changed joaat from 0x7DBCEF6F to 0x45268B6F.
0x7DBCEF6F was used to reset the calming quad counter to 0 on console. afaik it's not available on new builds.

Changed joaat from 0x7DBCEF6F to 0x45268B6F.
0x7DBCEF6F was used to reset the rise counter to 0 on console. afaik it's not available on new builds.
@fingaweg

Copy link
Copy Markdown

0x45268B6F = REMOVE_EXTRA_CALMING_QUAD

@stannum-cfx stannum-cfx added the needs validation This looks good, but needs additional confirmation of suggested change. label May 5, 2022
@stannum-cfx
stannum-cfx requested a review from blattersturm May 5, 2022 17:17
@stannum-cfx

Copy link
Copy Markdown
Contributor

I'm not sure how to verify if this won't break anything, @blattersturm any ideas?

@AvarianKnight

AvarianKnight commented Aug 9, 2024

Copy link
Copy Markdown
Collaborator

I don't think jooat values are used for anything besides documenting if a name is valid or not, which this also fixes. I'll double check to make sure this does break anything before merging

@AvarianKnight AvarianKnight added merging-blocked PR is waiting for something else to be merged. and removed needs validation This looks good, but needs additional confirmation of suggested change. labels Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merging-blocked PR is waiting for something else to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants