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

Wise bank import does not populate payee name #1328

Closed
russss opened this issue Jan 24, 2024 · 2 comments
Closed

Wise bank import does not populate payee name #1328

russss opened this issue Jan 24, 2024 · 2 comments
Milestone

Comments

@russss
Copy link
Member

russss commented Jan 24, 2024

Payee name was showing as "None" on the bank payment screen - these accounts were auto-populated by the Wise import function:

Additionally, the payment config screen (/admin/payment-config) does not show this problem.

We should:

  • Import the payee name (presumably this is possible) or require manual entry
  • Show the payee name on the payment config screen
  • Prevent a bank account from being activated on the payment config screen if it doesn't have a payee name.
  • Throw an exception in the payment flow if the payee name or any other required payment details are not present.

No issue for 2024, I've manually populated these in production.

@jayaddison
Copy link
Contributor

jayaddison commented Jan 24, 2024

Ok; we're pulling the incorrect field names here; the title field that we attempt to read from is from the new-format account-details API; previously the relevant field was, I think, named accountHolderName in the (now-deprecated) borderless-accounts API.

At the time of writing, the Account Details API isn't yet supported in pywisetransfer.

@jayaddison
Copy link
Contributor

Closing - I believe this was fixed by #1329.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants