-
Notifications
You must be signed in to change notification settings - Fork 89
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
Achievement cleanup #1529
Achievement cleanup #1529
Conversation
… exalted reputation achievements
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.
I would like "REMOVED_FROM_GAME" to be converted to proper timelines before merging
.contrib/Parser/DATAS/01 - Dungeons Raids/03 - Wrath of the Lich King/0.5 - Achievements.lua
Outdated
Show resolved
Hide resolved
…tions + some style stuff
…t wide achievements rather than in their main hub
…en to its own file
…moved to the raids where they belong
All so far is looking well, but wanna make sure that the main list reflect to the change for new Achievement layout/code, etc.. cuz i pull from git daily (i love reporting errors if i see one lol).. My achievement says 100% even tho with pop out says otherwise... |
Received permission via Discord DM to merge, so this can be checked on main in time for release
@vandiel01 Latest Git should show correctly for you, correct? |
yup so far! and thanks to you all for you do! |
Moved everything that was sourced into the achievement category to somewhere else in the addon so that section is finally its fully intended dynamic self. Open to feedback on where I've moved things. Else this will be merged in after a release, to allow some "live" review time by all contribs.