Skip to content

fix: ensure consistent outputPath type in dotcms-ui build configuration#34546

Merged
fmontes merged 2 commits intobring-back-modern-angular-builderfrom
copilot/sub-pr-34539
Feb 8, 2026
Merged

fix: ensure consistent outputPath type in dotcms-ui build configuration#34546
fmontes merged 2 commits intobring-back-modern-angular-builderfrom
copilot/sub-pr-34539

Conversation

Copy link
Contributor

Copilot AI commented Feb 7, 2026

Proposed Changes

  • Convert development configuration's outputPath from string to object format to match main configuration structure

Checklist

  • Tests
  • Translations
  • Security Implications Contemplated (no security impact)

Additional Info

The outputs array references {options.outputPath.base}, requiring outputPath to be an object. The development configuration was overriding it with a string, which breaks Nx output tracking/caching and may fail builder schema validation.

Before:

"development": {
    "outputPath": "../../tomcat9/webapps/ROOT/dotAdmin"
}

After:

"development": {
    "outputPath": {
        "base": "../../tomcat9/webapps/ROOT/dotAdmin",
        "browser": ""
    }
}

This maintains the same output path while ensuring type consistency across all build configurations.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Convert development configuration's outputPath from string to object format
to match the main configuration and ensure Nx output tracking works correctly.

Co-authored-by: fmontes <751424+fmontes@users.noreply.github.com>
Copilot AI changed the title [WIP] Work in progress to address feedback on angular builder for dotcms-ui fix: ensure consistent outputPath type in dotcms-ui build configuration Feb 7, 2026
Copilot AI requested a review from fmontes February 7, 2026 16:32
@fmontes fmontes marked this pull request as ready for review February 8, 2026 01:43
@fmontes fmontes merged commit a3c35a3 into bring-back-modern-angular-builder Feb 8, 2026
6 checks passed
@fmontes fmontes deleted the copilot/sub-pr-34539 branch February 8, 2026 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants