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

Class-restricted items erroneously trigger attempts for Dracthyr #774

Open
rdw-software opened this issue Sep 24, 2024 · 1 comment
Open

Comments

@rdw-software
Copy link
Member

Source: Discord

Toy - Skull of Corruption drop of Gul'dan in Nighthold. Rarity is tracking attempts with Dracthyr. As I understand it the only class this drops for is DH.

This is due to the nature of disableForClass being opt-out. Whenever a new class is added, all items with this setting have to be updated. Thankfully, there's only a few of them in the database, but this would also affect custom items. Maybe there's a way to avoid having to manually update these in the future, which is likely to be forgotten. If not, could at least add some unit tests or other reminders that make sure a mismatch between the list of supported classes and disableForClass settings doesn't survive the CI pipeline.

Affected items:

  • Skull of Corruption
  • Fandral's Seed Pouch
@rdw-software
Copy link
Member Author

The immediate problem has been fixed via #775. I'll leave this open because future desyncs are still possible without tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: The time is nigh (Planning)
Development

No branches or pull requests

1 participant