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

Add tracking for Box of Rattling Chains #358

Closed
godejord opened this issue Jul 1, 2021 · 6 comments · Fixed by #661
Closed

Add tracking for Box of Rattling Chains #358

godejord opened this issue Jul 1, 2021 · 6 comments · Fixed by #661

Comments

@godejord
Copy link
Collaborator

godejord commented Jul 1, 2021

https://www.wowhead.com/item=187075/box-of-rattling-chains
Need to detect which Torghast layer players are on. Only drops on layer 9-12

@nickbayley
Copy link
Contributor

Not sure if there is a better way, but GetInstanceInfo() exposes LfgDungeonID. For example 2197 is "Torghast: Skoldus Hall (Layer 9)".

@rdw-software
Copy link
Member

Any progress on this (since you self-assigned the issue)? If not, we could ask people on Discord if someone is still running Torghast?

@godejord
Copy link
Collaborator Author

Any progress on this (since you self-assigned the issue)? If not, we could ask people on Discord if someone is still running Torghast?

Unfortunately haven't gotten around to look into this. Will release the assignation and hope someone else have time to pursue it

@godejord godejord removed their assignment Sep 24, 2023
@rdw-software
Copy link
Member

No worries! I'll see if can hack together something that might just work, and then it's up to others to test it (I guess) 🚀

@cyriun
Copy link
Contributor

cyriun commented Sep 25, 2023

does this drop up to layer 16 now? seeing comments saying it dropped from layer 16

@rdw-software
Copy link
Member

I have no info beyond those wowhead comments, but it looks like it does drop from higher layers. Perhaps at the time this issue was created the maximum layer was lower, and it now includes all those above that? (would have to dig up patch notes to confirm)

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

Successfully merging a pull request may close this issue.

4 participants