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

Rename /transfers/* endpoint to /payments/* #2084

Closed
palango opened this Issue Aug 7, 2018 · 1 comment

Comments

Projects
None yet
2 participants
@palango
Collaborator

palango commented Aug 7, 2018

Problem Definition

In Tenerife we decided that a payment is The process of sending tokens from one account to another. May be composed of multiple transfers (Direct or HTL). A payment goes from Initiator to Target.

In #1949 we add a view into the payment history. This issue is to rename the current '/transfers/<hexaddress:token_address>/<hexaddress:target_address>' entpoint.

ToDo:

  • Rename POST '/transfers/<hexaddress:token_address>/<hexaddress:target_address>' to `POST '/payments/hexaddress:token_address/hexaddress:target_address'
  • Update the docs
  • Update the WebUI

@palango palango added this to the Red Eyes Testnet 4 milestone Aug 7, 2018

@palango palango self-assigned this Aug 7, 2018

palango added a commit to palango/raiden that referenced this issue Aug 7, 2018

@palango palango referenced this issue Aug 7, 2018

Closed

Revise Raiden Glossary #2030

0 of 2 tasks complete
@LefterisJP

This comment has been minimized.

Collaborator

LefterisJP commented Aug 7, 2018

In Tenerife we decided that

Please do not mention that as a reason to do anything. I'd rather see links to documentation. I recall that conversation but it's not well documented so I at least added that definition to the glossary: #2095
Note: It's also in the raiden network spec repo.

palango added a commit to palango/raiden that referenced this issue Aug 8, 2018

@palango palango closed this in #2085 Aug 8, 2018

palango added a commit that referenced this issue Aug 8, 2018

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