-
Notifications
You must be signed in to change notification settings - Fork 13
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
BfA fix #7
base: master
Are you sure you want to change the base?
BfA fix #7
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Honestly I think you should probably host this as your own update - mine were very small, just a few quick fixes for a friend for the sound file mid-Legion change. I don't use MSBT myself so I think it would make more sense for you to make your own fork, as it's clear you understand this addon better than I do and are more likely to keep it updated moving forward. Nice job!
MSBT would error out at MSBTProfiles.lua lines 1041 and 1047 with string concatenation errors when trying to load. Re-Adding L.MSG_AC and L.MSG_AC_FULL fixes the issue
Add missing expected L.MSG_AC and L.MSG_AC_FULL
Thanks for your suggestion. I'd maintain this one as possible as I can. But this project should be temporary because the author still maintain his own addons and it will be fixed soon by him. I'd do my best if you give permission to me even though I'm very noob in github. |
I've fixed MSBT for BfA 8.0