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.

Sign up for GitHub

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

Jump to bottom

WW-5455 Defines a new plugin to support Jasper Reports 7 #1124

Merged
merged 10 commits into from
Mar 6, 2025
Merged

WW-5455 Defines a new plugin to support Jasper Reports 7 #1124

merged 10 commits into from
Mar 6, 2025

Conversation

lukaszlenart
Copy link
Member

WW-5455

Jasper Reports 7 break backward compatibility with previous Jasper Reports, even JRXML file definition has changed.

@lukaszlenart lukaszlenart force-pushed the feature/WW-5455-jasperreports7 branch from 1b02dec to 7201bd9 Compare November 10, 2024 11:29
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
63.4% Coverage on New Code (required ≥ 80%)
21.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

Base automatically changed from release/struts-7-0-x to master November 30, 2024 07:55
@lukaszlenart lukaszlenart force-pushed the feature/WW-5455-jasperreports7 branch from 7201bd9 to 95915dc Compare November 30, 2024 09:23
@lukaszlenart lukaszlenart force-pushed the feature/WW-5455-jasperreports7 branch from 699154f to 9ed6485 Compare December 19, 2024 07:27
@lukaszlenart lukaszlenart added this to the 7.1.0 milestone Jan 5, 2025
@lukaszlenart lukaszlenart force-pushed the feature/WW-5455-jasperreports7 branch from 9253668 to 91d13bb Compare March 4, 2025 18:05
@lukaszlenart lukaszlenart marked this pull request as ready for review March 4, 2025 18:28
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Mar 4, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
69.4% Coverage on New Code (required ≥ 80%)
5.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

kusalk
kusalk approved these changes Mar 5, 2025
View reviewed changes
Copy link
Member

@kusalk kusalk left a comment

Choose a reason for hiding this comment

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

Not familiar with Jasper Reports to be able to review this in detail

@lukaszlenart
Copy link
Member Author

Not familiar with Jasper Reports to be able to review this in detail

Me too, and that's why I marked it as experimental tbh. Thanks!

@lukaszlenart lukaszlenart merged commit 1c3219e into main Mar 6, 2025
8 of 9 checks passed
@lukaszlenart lukaszlenart deleted the feature/WW-5455-jasperreports7 branch March 6, 2025 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kusalk kusalk kusalk approved these changes

@rgielen rgielen Awaiting requested review from rgielen

@aleksandr-m aleksandr-m Awaiting requested review from aleksandr-m

@yasserzamani yasserzamani Awaiting requested review from yasserzamani

@sdutry sdutry Awaiting requested review from sdutry

@cnenning cnenning Awaiting requested review from cnenning

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
7.1.0
Development

Successfully merging this pull request may close these issues.
2 participants
@lukaszlenart @kusalk