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

Aborting of in-progress push payments #473

Open
mattsaxon opened this issue Mar 23, 2017 · 3 comments
Open

Aborting of in-progress push payments #473

mattsaxon opened this issue Mar 23, 2017 · 3 comments

Comments

@mattsaxon
Copy link
Contributor

With respect to Roy's demo of push payments using the 'fb' PMI, I think it would be useful to more specifically define the behaviour of when Abort() is called when a payment in 'processing'.

In particular, I wonder if a specific state of "Processing" would be useful to define in the state machine in addition to "Created", "Interactive" and "Closed".

This would allow the definition of a specific exception to be thrown in the abort() algorithm to disambiguate the various reasons for failure, e.g. "AlreadyProcessing"

@mattsaxon
Copy link
Contributor Author

With regard to the proposal for a "processing" status, it is worth highlighting that section 16.10 explicitly calls out an addition state of pending, "the payment request user interface remains in a pending state". It is worth clarifiying what this pending state means and if it is part of the state model in section 3.10 (internal slot: state)

@mattsaxon
Copy link
Contributor Author

this issue is related to one in the payment handler, issue 117.

@zkoch
Copy link
Contributor

zkoch commented Aug 2, 2017

Marking as postponed since we don't have this in any current implementation, and we'd have to mark as "at-risk" for now anyway because of that. We'd prefer to postpone for now, though we think it's a good and useful idea.

@marcoscaceres marcoscaceres removed this from the CR milestone Aug 7, 2017
@marcoscaceres marcoscaceres changed the title Aborting of in progress push payments Aborting of in-progress push payments Jan 23, 2018
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