#1558: added new text about installation log path to installation dialog and exit dialog - #2076
Conversation
Coverage Report for CI Build 31185771056Warning No base build found for commit Coverage: 72.896%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsRequires a base build to compare against. How to fix this → Coverage Stats💛 - Coveralls |
56f9da6 to
18c1d7e
Compare
hohwille
left a comment
There was a problem hiding this comment.
@JoelAdbu thanks for your PR. Nice enhancement for our MSI installer. 👍
Did you test this? I am slightly confused by the [USERFOLDER].ide\logs syntax since in all existing places we are using [%USERPROFILE]\.ide\logs. Does the latter not work to stay homogenous and avoid such confusion?
Apart from some tiny format review comments ready for merge.
|
@hohwille, yes I tested it. %USERPROFILE% was shown literally in the dialog and was not resolved to the actual user directory. Therefore I switched to [USERFOLDER], which is populated via DirectorySearch and gets resolved correctly in the MSI UI. |
02e5ea4 to
d2997dd
Compare
… and exit dialog added doc about installation log path
- small fixes in path
- added msi installation log doc to documentation\setup.adoc - deleted msi installation log documentation from README.adoc
Co-authored-by: Jörg Hohwiller <hohwille@users.noreply.github.com>
Co-authored-by: Jörg Hohwiller <hohwille@users.noreply.github.com>
Co-authored-by: Jörg Hohwiller <hohwille@users.noreply.github.com>
d2997dd to
9829044
Compare
hohwille
left a comment
There was a problem hiding this comment.
@JoelAdbu thank you for your PR and improving our installer UX. 👍
Great that you also properly formatted our MSI XML files.
In this PR this makes the diff a little hard to read but in the future this will help us very much.
Since this was tested already in team review, I only checked the obvious changes and all looks good to me. Ready for merge.

This PR fixes #1558
Implemented changes:
Testing instructions
Please add conscise, understandable instructions on how a reviewer can test/verify the functionality of your contribution here:
Prerequisites:
native-imageis required to build the IDEasy native executable.documentation/contributing/GraalVM-Build-Guide.adocBuild the MSI package
windows-installer/README.adocVerify the installer UI
ideasy.msiVerify the installation log location:
%USERPROFLIE%\.ide\logsinstall-2026-07-01_10-15-00.logis created.Verify documentation
Checklist for this PR
Make sure everything is checked before merging this PR. For further info please also see
our DoD.
mvn clean testlocally all tests pass and build is successful#«issue-id»: «brief summary»(e.g.#921: fixed setup.bat). If no issue ID exists, title only.In Progressand assigned to you or there is no issue (might happen for very small PRs)with
internal