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

API discrepancies #38

Open
danielcondemarin opened this issue Jan 14, 2021 · 3 comments
Open

API discrepancies #38

danielcondemarin opened this issue Jan 14, 2021 · 3 comments

Comments

@danielcondemarin
Copy link

Hi 👋

According to the documentation the following fields are deprecated:

title (superseded by name)
first_name
bank_account_name

alternative_bank_account_names (superseded by alternative_names)

However those changes don't seem to be implemented in the docker api provided - form3tech/interview-accountapi:v1.0.0-4-g63cf8434. The fields name and alternative_names are being ignored.

Additionally the switched field doesn't persist in the database nor is returned from the API on account creation.

@dotchev
Copy link

dotchev commented Jan 27, 2021

may be this is the debugging exercise 😉

@irischeck
Copy link

irischeck commented Apr 2, 2021

Hi,

During account create op for UK account with minimal payload(non CoP) is returning 400 error with message
"account_classification in body should be one of [Personal Business]"

But according to the doc it's a non required param and only used for the CoP request.

This is creating a bit of confusion on required and non required data.

@irischeck
Copy link

I am not sure if this is the expected behaviour but I have figured what was the issue. I was testing the Create API without omitting empty fields and thus having "account_classification": "" in the payload.

Failed Payload: failed-payload.txt

According to the doc almost all the CoP fields are optional except the name field. Strangely, when I removed only the account_classification field it magically worked but I suspect that it shouldn't!

The required CoP name field that I have passed should have triggered a validation error as this is a required field and I have passed an array of empty strings!
"name": [ "", "", "", "" ]

Also, When I am passing the name and alternative_names correctly, they are not being set correctly.
Here is payload: req-resp-payload.txt

No account_number and iban are being generated. Also status field is missing in the response.

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

No branches or pull requests

3 participants