-
Notifications
You must be signed in to change notification settings - Fork 299
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
SEP-9 : Add external_transfer_memo
field to Financial Account Fields
#1452
Conversation
Hi @JakeUrban, After this change how do you feel about deprecating |
ecosystem/sep-0009.md
Outdated
@@ -109,6 +109,7 @@ applications use fields that are the most familiar, which are often specific to | |||
| `bank_number` | string | Number identifying bank in national banking system (routing number in US) | | |||
| `bank_phone_number` | string | Phone number with country code for bank | | |||
| `bank_branch_number` | string | Number identifying bank branch | | |||
| `memo` | string | A destination tag/memo used to identify a transaction | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WDYT about renaming it to external_transfer_memo
?
So it's not confused with Stellar's transaction memo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That sounds good to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool! It is updated.
dc66a19
to
34eefcc
Compare
@ydag could you update version and changelog of the SEP-9 file? |
memo
field to Financial Account Fieldsexternal_transfer_memo
field to Financial Account Fields
Thanks for proposing this change @ydag 🎉 |
Resolving the issue #1451