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

why do you need to provide a mandat for a money transfer? What is a debtorId? #66

Open
i-wahdan opened this issue Mar 7, 2024 · 2 comments · May be fixed by #68
Open

why do you need to provide a mandat for a money transfer? What is a debtorId? #66

i-wahdan opened this issue Mar 7, 2024 · 2 comments · May be fixed by #68

Comments

@i-wahdan
Copy link

i-wahdan commented Mar 7, 2024

Hi, I am trying to check the functionality of the library, but I have no idea what kind of information is needed and why.

For example, a mandate in SEPA is only needed for a direct debit, why must it be defined for a money transfer?
Also why is Mandate Signature Date not an optional field where it is not required from actual banks?
Also I know what is a creditorId for a direct debit, but no where in the internet is there a reference of such thing as debtorId. What is this supposed to be?

I hope someone is still there maintaining the library who can check these questions. Thanks!

@kewisch
Copy link
Owner

kewisch commented Apr 24, 2024

I'm still maintaining the library, though honestly I never ended up using it in production so I can't answer more specific questions. I went by my understanding of the spec back then. I'm happy to accept changes and would appreciate pointers to the spec when doing so.

@fellmann fellmann linked a pull request Jul 5, 2024 that will close this issue
@fellmann
Copy link

fellmann commented Jul 5, 2024

To my limited understanding:

  • the mandate is only required for direct debit, I have removed the check for transfers
  • the debitor id is not required, and incorrectly serialized at the moment.

So, mandate id, mandate signature date and debitor id should not be set for transfer.

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

Successfully merging a pull request may close this issue.

3 participants