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

Disenchant item BUG #49

Closed
kuramajssken001 opened this issue Nov 30, 2018 · 3 comments
Closed

Disenchant item BUG #49

kuramajssken001 opened this issue Nov 30, 2018 · 3 comments

Comments

@kuramajssken001
Copy link

Last commit

https://youtu.be/MU7znVlr66k

look this @ratkosrb thanks.

@ratkosrb
Copy link
Contributor

I think it was made to auto loot on purpose to prevent a dupe exploit.

@kuramajssken001
Copy link
Author

I think it was made to auto loot on purpose to prevent a dupe exploit.

Admire you @ratkosrb

@CansecoDev
Copy link

CansecoDev commented Nov 30, 2018

Wouldn't that introduce a new issue on the use case where the enchanter has its bags full?

I don't know (I haven't tested that), just guessing here, but if that's the case, the normal behaviour should be to let the materials stay on the container until the enchanter picks it up so he can make space on his bag if they are full.

Velite12 pushed a commit to Velite12/core that referenced this issue Oct 11, 2021
* Remove unused text

It is not used by this script, but later defined and used properly in ragnaros script file.

* Use the already defined text
spirehawk pushed a commit to spirehawk/westfall that referenced this issue Mar 21, 2022
* Misc DB fixes

* Darnassus banker faction
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

No branches or pull requests

3 participants