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

Ticket sending #127

Merged
merged 16 commits into from
Feb 3, 2024
Merged

Ticket sending #127

merged 16 commits into from
Feb 3, 2024

Conversation

KwikKill
Copy link
Member

This PR :

  • Deref ticket in tournament/me in order to get the ticket id
  • Change ticket /get/{username}/{token} to /get/{id}/{token}
  • Add a generate ticket pdf from ticket method
  • Add a send ticket by mail method
  • Send ticket on payment

The ticket generation use reportlab. This is pretty hard to maintains imo but no serious alternatives were available. I tried a bunch of tools to convert html to pdf so we could had only a html template but they were all not supporting CSS3 or based on webdrivers hard to maintains.

@KwikKill KwikKill added enhancement help wanted Extra attention is needed labels Jan 25, 2024
Copy link
Contributor

@Lugrim Lugrim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only the sanitation is actually mandatory, my other suggestions mostly are for code quality

@KwikKill KwikKill requested a review from Lugrim January 25, 2024 09:51
@KwikKill KwikKill self-assigned this Jan 25, 2024
Copy link
Contributor

@Lugrim Lugrim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAIK my concerns have been resolved, good job!

Copy link
Contributor

@Lymkwi Lymkwi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one nitpick, but it could be fixed in #128 .

@KwikKill KwikKill merged commit a19d19a into dev Feb 3, 2024
4 checks passed
@KwikKill KwikKill deleted the ticket-sending branch February 3, 2024 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants