Skip to content

Standarized overwrite handling by export multi file - #2631

Open
TruongQuangSB wants to merge 4 commits into
mainfrom
document-export-overwrite-multi
Open

Standarized overwrite handling by export multi file#2631
TruongQuangSB wants to merge 4 commits into
mainfrom
document-export-overwrite-multi

Conversation

@TruongQuangSB

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actionsBot commented Aug 20, 2026

Copy link
Copy Markdown

Test Results

147 tests ±0 147 ✅ ±0 3m 31s ⏱️ +18s
40 suites ±0 0 💤 ±0 
40 files ±0 0 ❌ ±0 

Results for commit ae1c4b2. ± Comparison against base commit f83847d.

This pull request removes 4 and adds 4 tests. Note that renamed tests count towards both.
org.eclipse.set.feature.plazmodel.test.CalculateTopologicalCoordinateTest ‑ testTopologischeCoordinateCalculate(Pair)[1] Pair [first=pphn, second=/tmp/set-test1055487389374426939PPHN_1.10.0.3_01-02_Ibn-Z._-_2._AeM_2022-05-17_13-44_tg3.planpro]
org.eclipse.set.feature.siteplan.transform.SiteplanTransformatorTest ‑ testSiteplanTransformSuccessful(String)[1] /tmp/set-test18239906140177117608PPHN_1.10.0.3_01-02_Ibn-Z._-_2._AeM_2022-05-17_13-44_tg3.planpro
org.eclipse.set.feature.table.pt1.test.Pt1TableTransformationTest ‑ testTransformator(String)[1] /tmp/set-test3004754087439147612PPHN_1.10.0.3_01-02_Ibn-Z._-_2._AeM_2022-05-17_13-44_tg3.planpro
org.eclipse.set.feature.table.pt1.test.Pt1TableTransformationTest ‑ testTransformator(String)[2] /tmp/set-test2508624512478775280Info__2026-04-21_10-40.planpro
org.eclipse.set.feature.plazmodel.test.CalculateTopologicalCoordinateTest ‑ testTopologischeCoordinateCalculate(Pair)[1] Pair [first=pphn, second=/tmp/set-test13474606634918194612PPHN_1.10.0.3_01-02_Ibn-Z._-_2._AeM_2022-05-17_13-44_tg3.planpro]
org.eclipse.set.feature.siteplan.transform.SiteplanTransformatorTest ‑ testSiteplanTransformSuccessful(String)[1] /tmp/set-test2198156192233394224PPHN_1.10.0.3_01-02_Ibn-Z._-_2._AeM_2022-05-17_13-44_tg3.planpro
org.eclipse.set.feature.table.pt1.test.Pt1TableTransformationTest ‑ testTransformator(String)[1] /tmp/set-test16853007091396083834PPHN_1.10.0.3_01-02_Ibn-Z._-_2._AeM_2022-05-17_13-44_tg3.planpro
org.eclipse.set.feature.table.pt1.test.Pt1TableTransformationTest ‑ testTransformator(String)[2] /tmp/set-test7421930208450027976Info__2026-04-21_10-40.planpro

♻️ This comment has been updated with latest results.

TruongQuangSBand others added 2 commits August 25, 2026 09:43
…/feature/table/export/PlanProExportPart.java
Co-authored-by: Marius Heine <m.heine@geprog.com>
@mariusheine

Copy link
Copy Markdown
Contributor

@TruongQuangSB A success message is missing in the Document and PlanPro Export parts. The tableoverview have this success message

@TruongQuangSB

Copy link
Copy Markdown
ContributorAuthor

@TruongQuangSB A success message is missing in the Document and PlanPro Export parts. The tableoverview have this success message

@mariusheine hmm, when it only don't have table to export, then given't "success" dialog

@mariusheine

Copy link
Copy Markdown
Contributor

@TruongQuangSB A success message is missing in the Document and PlanPro Export parts. The tableoverview have this success message

@mariusheine hmm, when it only don't have table to export, then given't "success" dialog

I actually would like to see the success message because I clicked "OK" on an export dialog and and don't get a response if it is finished or not. If there really are no files to export, then we should maybe show a message "No data to export"

@TruongQuangSB

Copy link
Copy Markdown
ContributorAuthor

@TruongQuangSB A success message is missing in the Document and PlanPro Export parts. The tableoverview have this success message

@mariusheine hmm, when it only don't have table to export, then given't "success" dialog

I actually would like to see the success message because I clicked "OK" on an export dialog and and don't get a response if it is finished or not. If there really are no files to export, then we should maybe show a message "No data to export"

ok

Sign up for freeto 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.

2 participants

@TruongQuangSB@mariusheine