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

Update branch to master #1445

Merged
merged 39 commits into from
Jun 25, 2023
Merged

Update branch to master #1445

merged 39 commits into from
Jun 25, 2023

Conversation

Sluimerstand
Copy link
Collaborator

No description provided.

ImUnicke and others added 30 commits June 23, 2023 18:07
…ATT from saving Tournament data into the user's Saved Variables and potentially contaminating their cached data
…pecially when the Contains greatly exceedes the limits)

GetCachedSearchResults now pulls the relative difficultyID of the root result into the result itself for proper filtering of any filled content (i.e. ToT boss tooltips showing CBD from all difficulties) (when a search result contains multiple initial results, it automatically clears all parent references)
* Event-driven Updates when collecting a 'Thing' now properly refresh the respective 'costs' (if any) recursively via iterative updates
* Fixed popouts of needed costs not being accurate when the cost becomes finished
TopLevelUpdateGroup now uses UpdateGroup internally to ensure accurate calculations from all possible sources
…t un-related to the group (i.e. Things dropped from many creatures showing the drops from the creatures under the respective Things)
TopLevelUpdate now does the proper OnUpdate sequence
Fixed an Aberrus item which shows the same appearance on all difficulties in dungeon journal (Blizzard pls fix)
…ments in TBC/Wrath, but becomes available to both with Cataclysm.
ImUnicke and others added 9 commits June 24, 2023 21:10
Added 'OnReady' processing into ATT (allows modules to post-set their own locals with static content defined in ATT.lua)
ATT now hooks into Tooltips after ATT is fully ready to support tooltip hooks
…c' (after testing various means, I could not find a way to obtain this quest on a character which already has a Garrison. Party Sync/Chromie Time/View Garrison, no combination worked)

Couple other comments and adjustments from testing
@Sluimerstand Sluimerstand merged commit 931a505 into settingsV3 Jun 25, 2023
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

Successfully merging this pull request may close these issues.

6 participants