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

TODO: Achievements #8

Closed
heyitsbench opened this issue Mar 7, 2022 · 3 comments
Closed

TODO: Achievements #8

heyitsbench opened this issue Mar 7, 2022 · 3 comments
Labels
enhancement New feature or request

Comments

@heyitsbench
Copy link
Owner

No description provided.

@heyitsbench heyitsbench added the enhancement New feature or request label May 23, 2022
@heyitsbench
Copy link
Owner Author

Realm first level 80 achievements look to be easily done in achievement.dbc, whereas the know thy enemy achievement needs modifying in achievement_criteria.dbc. If there are any other achievements that need adjustment/adding, please let me know. Otherwise, should be committed today.

Also of note, if #7 happens, then this will need to be reopened to add the exploration achievements for the respective zones.

@heyitsbench
Copy link
Owner Author

heyitsbench commented Jun 18, 2022

Looks like Let It Snow, Fistful of Love, Check Your Head, and Shake Your Bunny-Maker are also achievements requiring changes. Ugh.

EDIT: In keeping with my pseudo-blizzlike stance, apparently Fistful of Love and Let It Snow don't require changes, as their criteria was not changed in any way for Cata. ¯\(ツ)

@heyitsbench
Copy link
Owner Author

Closed with 649f086 and 9b9e377. Hallow's End masks will need to be made for those achievements, but that'll be horrible so I don't want to yet.

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

No branches or pull requests

1 participant