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

https://github.com/alfio-event/alf.io/issues/1125 #1303

Open
wants to merge 223 commits into
base: 2.0-M4-maintenance
Choose a base branch
from

Conversation

sergyer
Copy link
Contributor

@sergyer sergyer commented Dec 1, 2023

Added voucher code in attendees data.

cbellone and others added 30 commits January 31, 2022 15:14
* enable accessibility for pdf ticket

* add metadata for credit-note/invoice/receipt
alfio-event#1067)

* initial work for removing log4j2 lombok annotation

* continue removal log4j2 annotation work

* continue removal log4j2 annotation work

* continue removal log4j2,slf4j annotation work

* remove utilityclass lombok annotation

* fix some errors in the build

* misc cleanup
* alfio-event#1068 remove sneakythrow

* alfio-event#1068 initial work for removing AllArgsConstructor
cbellone and others added 17 commits May 9, 2023 19:07
* api key system components working

* api-key-system bulk

* apikey bulk file

* Disable button if not valid form

* Remove console.log
* Removed default message api key works from template

* creat component event-item already takes the even from dashboad, img resposnsive

* optimize image loading

* UI event, load events, sort, active / past

* Crate modal UI

* Add logic for date

* Validation date on export

* Improved style for event title and pending payments logic

* Remove not from ngif condition

* Remove detail button
* Added docs for Mollie

Added a simple guide how to configure Mollie for use in Alf.io as payment method.

* Added section supported payment methods
* Fix button cancel from export modal

* Added cards

* Clean

* Add ng2-charts library

* Fixed cards using cols

* Improve event detail view

* Added panel with even info and language description

* Logic pieChart

* Added ticketStatistics call and model

* Added lineChart logic and formated data user friendly

* Added pipe and logics for cards event

* Added a few menu voices

* various fixes
* begining categories

* translate api-key list, user system, organizations

* translate event dashboard
* list menu with icons

* Added pipes, progress bar and check box logic part 1

* logic progress bar

* Fixed pipe for active and expire

* translate

* fixed button's esthetic

* fixed icons from menu

* fixed some classes

* added empty component ticket category detail

* fixed svg from button

* various fixes pr

* fixed typos
@sergyer
Copy link
Contributor Author

sergyer commented Dec 1, 2023

@syjer Please review.

@syjer
Copy link
Member

syjer commented Dec 2, 2023

Hi @sergyer , thank you for the PR. Everything seems ok.

Can I ask you to change the pull request so the target branch is 2.0-M4-maintenance instead of master? We are currently trying to reconcile the two branches, so for simplicity reasons, all small changes should first go on the 2.0-M4-maintenance branches.

Thank you :)

@sergyer sergyer changed the base branch from master to 2.0-M4-maintenance December 6, 2023 09:34
@sergyer sergyer closed this Dec 6, 2023
@sergyer sergyer reopened this Dec 6, 2023
@sergyer
Copy link
Contributor Author

sergyer commented Dec 6, 2023

@syjer Sure. Done.

@syjer
Copy link
Member

syjer commented Dec 6, 2023

Hi @sergyer , I guess a cherry-pick of your commit on top of the 2.0-M4-maintenance would have been better, as currently this PR cannot be merged, as the branches have diverged too much. (You may have noticed the the impossibility to merge you PR due to the amount of changed files outside your change).

@@ -41,7 +42,7 @@

public static void sendHeaders(HttpServletResponse response, String eventName, String reservationId, BillingDocument billingDocument) {
response.setHeader("Content-Disposition", "attachment; filename=\"" + getBillingDocumentFileName(eventName, reservationId, billingDocument) + "\"");
response.setContentType("application/pdf");
response.setContentType(MediaType.APPLICATION_PDF_VALUE);

Check warning

Code scanning / CodeQL

HTTP response splitting Medium

This header depends on a
user-provided value
, which may cause a response-splitting vulnerability.
This header depends on a
user-provided value
, which may cause a response-splitting vulnerability.
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

Successfully merging this pull request may close these issues.

6 participants