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

auto_sync.py fails to start: notification_filters was moved #15

Closed
zr40 opened this issue Oct 27, 2019 · 2 comments

Comments

@zr40
Copy link
Contributor

@zr40 zr40 commented Oct 27, 2019

On October 8th, bunq has removed notification_filters from the MonetaryAccount object and introduced separate endpoints: https://beta.doc.bunq.com/basics/changelog#october-8-2019

Due to this change, auto_sync.py now fails to start with the following error:

Traceback (most recent call last):
  File "auto_sync.py", line 101, in <module>
    add_callback(port)
  File "auto_sync.py", line 49, in add_callback
    "notification_target": url
  File "auto_sync.py", line 59, in set_autosync_callbacks
    old_nfs = bunq_api.get_callbacks(bunq_user_id, bunq_account_id)
  File "/home/bunq2ynab/bunq2ynab/bunq_api.py", line 34, in get_callbacks
    return result[account_type]["notification_filters"]
KeyError: 'notification_filters'
@wesselt

This comment has been minimized.

Copy link
Owner

@wesselt wesselt commented Oct 27, 2019

Thanks for letting me know. My auto sync was still running fine, perhaps because it was started before this was changed.

I've updated auto_sync and it should now work with the new endpoints. Works for me, how does it work for you?

@zr40

This comment has been minimized.

Copy link
Contributor Author

@zr40 zr40 commented Oct 28, 2019

Thanks for the update. It works fine for me :)

@zr40 zr40 closed this Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.