Allow payments to be cancelled #22

Closed
marksteward opened this Issue May 29, 2012 · 2 comments

Comments

Projects
None yet
2 participants
@marksteward
Owner

marksteward commented May 29, 2012

This will help with the per-user ticket limit.

Don't delete the payment, but set its status to "cancelled". Reset the expiry on the tickets to a couple of hours and tell the user so they can create a new payment.

@JasperWallace

This comment has been minimized.

Show comment Hide comment
@JasperWallace

JasperWallace Jun 2, 2012

Member

we have 2 cases:

a) deleting 'new' gocardless payments:

I think we can just delete them, no money has changed hands, should log that we've done it tho.

b) canceling 'inprogress' gocardless payments

In this case money will be coming to us, and we have to take into acount the £10 admin fee... I think this needs to be part of the admin interface

c) inprogress banktransfers

We need to keep the payment around (but hide it in the ui?) so that if the user does use the reference for the payment we can work out where it came from.

Member

JasperWallace commented Jun 2, 2012

we have 2 cases:

a) deleting 'new' gocardless payments:

I think we can just delete them, no money has changed hands, should log that we've done it tho.

b) canceling 'inprogress' gocardless payments

In this case money will be coming to us, and we have to take into acount the £10 admin fee... I think this needs to be part of the admin interface

c) inprogress banktransfers

We need to keep the payment around (but hide it in the ui?) so that if the user does use the reference for the payment we can work out where it came from.

@JasperWallace

This comment has been minimized.

Show comment Hide comment
@JasperWallace

JasperWallace Jun 2, 2012

Member

for a) above we'd need:

A form + 'cancel' submit button on /tickets, which POST's to a /cancel/gocardless (to avoid csrf) which has a yes/no prompt, Yes then deletes+logs the payment + ticket(s), No goes back to /tickets.

Member

JasperWallace commented Jun 2, 2012

for a) above we'd need:

A form + 'cancel' submit button on /tickets, which POST's to a /cancel/gocardless (to avoid csrf) which has a yes/no prompt, Yes then deletes+logs the payment + ticket(s), No goes back to /tickets.

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