Skip to content

Some new features and fixes#19

Open
ruscon wants to merge 4 commits intom3talstorm:masterfrom
ruscon:master
Open

Some new features and fixes#19
ruscon wants to merge 4 commits intom3talstorm:masterfrom
ruscon:master

Conversation

@ruscon
Copy link
Copy Markdown

@ruscon ruscon commented Dec 3, 2017

I don't know python, but it's working, u can check :)

@druban
Copy link
Copy Markdown

druban commented Jan 27, 2018

There's a bug in the hidden reward code. When it picks up a hidden reward, it doesn't clear it, meaning that in the next account refresh cycle it'll attempt to collect it again resulting in "internal error" message.

@druban
Copy link
Copy Markdown

druban commented Mar 26, 2018

At the end of collect() method, before return, add:
# Clean up. Otherwise entry will be picked up on the next reload resulting in server error
self.session.query(HiddenReward).filter_by(hiddenRewardId=self.hiddenRewardId).delete()

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.

2 participants