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

Reservation detail enhancements #541

Closed
3 tasks done
cbellone opened this issue Nov 5, 2018 · 2 comments
Closed
3 tasks done

Reservation detail enhancements #541

cbellone opened this issue Nov 5, 2018 · 2 comments

Comments

@cbellone
Copy link
Member

cbellone commented Nov 5, 2018

Some enhancements suggested by event organizers:

  • send tickets to one or more attendees
    change the behavior of the current button, open a dialog to select which ticket(s) to send. Default selection: all

  • add email log, with the emails sent for the reservation
    this way it would be easier to check if an email has already been sent or not

  • add check-in log for each ticket (popover? list?)

@syjer
Copy link
Member

syjer commented Jan 8, 2019

note: I'm taking care of the email log in the reservation.

Will need to add a new column reservation_id to email_message (not a fk, nullable).

In term of UI, as only the new emails will have the reservation_id -> if the reservation has no email log, it means it's an "old style" one -> we show a message. In the other cases, we can show a message that warn that not all the emails will be attached to it.

syjer added a commit that referenced this issue Jan 22, 2019
…ble email sent on reservation confirmation and fix inverted parameter
cbellone pushed a commit that referenced this issue Jan 23, 2019
* #541 add reservation_id in email_message, modify logic and test

* #541 expose api to see emails sent to a specific reservation, fix double email sent on reservation confirmation and fix inverted parameter

* #541 expose the email log in the reservation ui
cbellone added a commit that referenced this issue Jan 23, 2019
@cbellone
Copy link
Member Author

implemented.

cbellone added a commit that referenced this issue Jan 24, 2019
syjer pushed a commit that referenced this issue Jan 24, 2019
* #541 remove unused tag

* #585 - send email automatically + add flag for control it
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

2 participants