-
Notifications
You must be signed in to change notification settings - Fork 1
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
Multi language support #35
Comments
Yeah I know I broke localisation, but it actually came in the back of the
fact the GetToyInfo method not working correctly more than half the time,
leading to people's hs not loading, not changing or only a rotating through
a couple of them.
…On Sun, 17 Nov 2024 at 15:33, BlartVersenwald ***@***.***> wrote:
toyName must be localized string otherwise macro would work only in
english language.
I believe it should be as simple as this
-- Randomly pick one.
local itemID = RandomKey(UsableHearthToyIndex)
local _, toyName = C_ToyBox.GetToyInfo(itemID)
—
Reply to this email directly, view it on GitHub
<#35>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGK64SO6PFCPLTYL6KWQKT2BCZNXAVCNFSM6AAAAABR6BPVWGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGY3DMMBTGM2DQMQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Simple change above works for me perfectly (english/cyrillic client) |
Likely the toy info hasn't loaded when you're calling that, so it's returning null. |
toyName must be localized string otherwise macro would work only in english language.
I believe it should be as simple as this
The text was updated successfully, but these errors were encountered: