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

[SOD] Something is wrong with the 'icon' code in CombatEvents.lua making the app crash #108

Closed
Larzie opened this issue Apr 9, 2024 · 3 comments

Comments

@Larzie
Copy link

Larzie commented Apr 9, 2024

Since the new phase (phase 3). Something is wrong with the icon code in CombatEvents.lua. The codechanges below make the addon work again, but it removes the icons from the scroll areas ofc.

Change line 2066 from this:
icon = icon(info)

To this:
-- icon = icon(info)

Do you see why icon(info) doesn't work anymore?

Thank you so much <3

@nteetor
Copy link

nteetor commented Apr 11, 2024

I have experienced this as well. It's not consistent for me and a /reload does reset the problem. I can only guess trying to look up a certain icon from something new to Phase 3 causes the error.

@nebularg
Copy link
Owner

nebularg commented Apr 12, 2024

Thanks, I'll push some quick fixes for this and work on combining the classic/retail version since it looks like a lot of the cruft needed for classic era has gone away (and is now causing problems)

nebularg added a commit that referenced this issue Apr 12, 2024
Resolves #94
Resolves #108
@nteetor
Copy link

nteetor commented Apr 14, 2024

Awesome, thank you. I'll reply if I see the problem again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants