woudt / bunq2ifttt

bunq2IFTTT creates a self-hosted interface between the bunq banking API and IFTTT.
MIT License
24 stars 8 forks source link

Add account name to IFTTT Ingredients #4

Closed jdambacher closed 4 years ago

jdambacher commented 4 years ago

It would be nice if the account name (name of the account in bunq) could be added as a ingredient. Currently it's only possible to get the account IBAN (IBAN of the account in bunq).

I'm also using the ifttt2ynab service and it's not really comfortable to have the account IBAN as account name in YNAB.

woudt commented 4 years ago

Thanks for the request. This shouldn't be a big problem. I'll have a look and will let you know as soon as it is done!

jdambacher commented 4 years ago

@woudt Thanks for the fast response!

woudt commented 4 years ago

@jdambacher No problem.

This has just been fixed in release v2.3. I still need to update the documentation, but the slug 'account_name' will give you the account name.

jdambacher commented 4 years ago

Thanks, I'll try it!

woudt commented 4 years ago

Documentation is now here: https://github.com/woudt/bunq2ifttt/wiki/Configure-payment-request-action

I am closing this issue now, though let me know if you encounter any problems or if you have any other suggestions for new features.

jdambacher commented 4 years ago

@woudt I've tried 'account_name' but it's still using the IBAN. I've added 'account_name' to the ingredients and changed the applet to use the account_name instead of the IBAN to get the right account in YNAB.

woudt commented 4 years ago

@jdambacher Hmm, that is weird, as it is working for me.

jdambacher commented 4 years ago

@woudt

  1. Yes
  2. Yes
  3. No, the field is completely empty.
woudt commented 4 years ago

@jdambacher Hmm, very weird.

Are you sure your upgrade of bunq2IFTTT to release 2.3 was successful? The API tests in IFTTT, do they all succeed?

If the answer is yes to both questions above, the only next step I see is to add some debugging code, so let me know.

woudt commented 4 years ago

@jdambacher Futhermore, in the log file there should be a line that starts with '[bunqcb_mutation] translated:'. In the json string after that, is the account_name field present and filled?

jdambacher commented 4 years ago

I've re-deployed the latest version and will try it again. I can find the mutation in the logs, but the account_name field is not present.

jdambacher commented 4 years ago

Ahhh ... maybe I found the issue. I was deploying both YNAB and BUNQ integration to the same Google Cloud App. I modified the app.yaml to set the service name for each application. After updating your code (for the BUNQ integration), I forgot the re-add the service declaration to the app.yaml. I'll try it now and will inform you on my next transaction.

jdambacher commented 4 years ago

It's working now! I think the issue was the service name.

Thanks for your support!