eve-val/evelink

corp wallet_journal is missing the accountKey parameter

bastianh opened this issue · 5 comments

the new

     def wallet_journal(self, before_id=None, limit=None, api_result=None):

in corp.py is missing the accountKey parameter to select the wallet

old:

    def wallet_journal(self, before_id=None, limit=None, accountKey=None):

Good catch, that is indeed a bug.

My hands are clean.
I don't think it was ever there before...

99fffb6#diff-7778925421062d9dde3ff3181e2ab25e <-- original method

2f5668f#diff-7778925421062d9dde3ff3181e2ab25eL169 <-- recent change

Yeah, I think it just got missed when the method was originally added.

strange.. I must have fixed it in my local copy then some time ago... I use a modified version that I made py3 compatible.

Anyhow, about to push a pull request that fixes this. One moment...