Skip to content
This repository was archived by the owner on Mar 12, 2022. It is now read-only.

Fixed pin button and filter issues#8

Merged
nitrog0d merged 9 commits into
nitrog0d:masterfrom
YukiAyase:master
Jun 26, 2020
Merged

Fixed pin button and filter issues#8
nitrog0d merged 9 commits into
nitrog0d:masterfrom
YukiAyase:master

Conversation

@YukiAyase

Copy link
Copy Markdown
Contributor

I went ahead and at least tried fixing the pins button to not make the game crash on click
Implementing this made filters not work again so I had my friend help me and fixed this, filters are fully working
Also found a method to be broken, my friend figured out a way to fix this

Made it so that pins button in camera menu does not crash, I guess
Made it so that clicking on pins button in camera menu not crash the game, I guess
Weird error while compiling, made a tiny change to it, I guess
Fixed crash on pin button when clicking
Fixed filters, for some reason it won't work when method number is changed
Changed method for 
GetVRCUiManager().Method_Public_Void_String_0(message);
to
GetVRCUiManager().Method_Public_Boolean_String_2(message);
For some reason old method was broken [thanks for my friend on helping me with this one]
Comment thread Utils/VRCUtils.cs
public static void QueueHudMessage(string message) {
GetVRCUiManager().Method_Public_Void_String_2(message);
// Old method was somehow broken [thanks to my friend for helping me with this]
GetVRCUiManager().Method_Public_Boolean_String_2(message);

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I suppose the correct method is Method_Public_Void_String_3, I'll change it, don't worry

@nitrog0d

Copy link
Copy Markdown
Owner

Thanks <3 I promise that in the next update I'll make it so it doesn't need to update the methods every update, not having much time right now...

@nitrog0d
nitrog0d merged commit c6aceff into nitrog0d:master Jun 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants