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

Removing attendees from a reservation doesn't update the invoice #410

Closed
salmar opened this issue Apr 4, 2018 · 4 comments
Closed

Removing attendees from a reservation doesn't update the invoice #410

salmar opened this issue Apr 4, 2018 · 4 comments
Assignees
Milestone

Comments

@salmar
Copy link
Contributor

salmar commented Apr 4, 2018

When an attendee is removed from a reservation (and refund is done), the invoice should be updated accordingly.

@cbellone
Copy link
Member

cbellone commented Apr 6, 2018

Hi @salmar, thanks for opening this issue.

As per #407, starting from 1.14 alf.io will always update the invoice/receipt in case of PENDING reservation modifications.

We (@syjer and I) are not entirely sure that updating a confirmed (and paid) reservation should always trigger an update on the invoice, since in some countries this is forbidden.
For example in Italy the invoice must remain the same, and you must issue a "Credit Note" for the refunded amount.

So it's not "safe" to update the invoice as default behavior.

We were thinking of adding an extra checkbox "Update Invoice" to the "Remove Ticket" view:

image

and then to add a row in the invoice, stating that a refund has been issued, something like

Refund -100.00 CHF

wdyt?

@cbellone
Copy link
Member

cbellone commented Apr 7, 2018

implementation can be considered done.
@mg-1999 could you please provide a translation for the following text?

This invoice has been updated after the cancellation of one or more tickets and the refund of {0}.

Thank you very much!

cbellone added a commit that referenced this issue Apr 8, 2018
@cbellone
Copy link
Member

cbellone commented Apr 9, 2018

closing this as it is fixed and we're going to release 1.14.
Will add Dutch translation as soon as possible.

@cbellone cbellone closed this as completed Apr 9, 2018
@cbellone cbellone added this to the 1.14 milestone Apr 9, 2018
@mg-1999
Copy link
Contributor

mg-1999 commented Apr 9, 2018

Translated text:
Dit factuur is geupdate na de annulering van een of meer tickets en de terugbetaling van {0}.

cbellone added a commit that referenced this issue Apr 10, 2018
syjer pushed a commit that referenced this issue Apr 11, 2018
syjer pushed a commit that referenced this issue Apr 11, 2018
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

No branches or pull requests

3 participants