search

wesselt / bunq2ynab

Upload bunq transactions to ynab
Other
78 stars 22 forks source link

Only syncing one out of 2 accounts. #28

Closed javydekoning closed 4 years ago

javydekoning commented 4 years ago

In my config I have 2 accounts, syncing to different budgets. This has stopped working, and I'm not sure why.

Opening a ticket for myself to research why this is happening.

wesselt commented 4 years ago

I added a parameter for ynab_budget_name, might have something to do with it?

javydekoning commented 4 years ago

Thanks, I'll look into it now. Any reason you hardcoded the ssmpath instead of using env vars?

wesselt commented 4 years ago

Yeah, I didn’t understand how to solve the “recursive” error without removing the env vars. That might be solved now with the HttpApi as its own resource.

Just remembered an issue where a “return” made the synch stop after the first account, here https://github.com/wesselt/bunq2ynab/commit/46d07c604377507d91a363ae989f8380dca4a59b

javydekoning commented 4 years ago

Yes, that commit fixed it.

The circular/recursive error is indeed because the HttpApi was part of the Function. It needs to create the API first before you can reference it. With the template like this, it should work.

I'll keep it running like this for a little bit again, if it's successful I'll publish to Serverless Application Repository and write new instructions.