Skip to content

[YUNIKORN-3214] Deploy browser content to root output dir#250

Closed
craigcondit wants to merge 1 commit intoapache:masterfrom
craigcondit:YUNIKORN-3214
Closed

[YUNIKORN-3214] Deploy browser content to root output dir#250
craigcondit wants to merge 1 commit intoapache:masterfrom
craigcondit:YUNIKORN-3214

Conversation

@craigcondit
Copy link
Contributor

What is this PR for?

After upgrading to Angular 20, the default output directory has a "browser" prefix prepended. Remove this to ensure content matches previously generated app.

What type of PR is it?

  • - Bug Fix
  • - Improvement
  • - Feature
  • - Documentation
  • - Hot Fix
  • - Refactoring

Todos

  • - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/YUNIKORN-3214

How should this be tested?

Manually verified that application deploys properly when deployed with scheduler.

Screenshots (if appropriate)

Questions:

  • - The licenses files need update.
  • - There is breaking changes for older versions.
  • - It needs documentation.

After upgrading to Angular 20, the default output directory has a
"browser" prefix prepended. Remove this to ensure content matches
previously generated app.
@craigcondit
Copy link
Contributor Author

Test failure is unrelated.

Copy link
Contributor

@wilfred-s wilfred-s left a comment

Choose a reason for hiding this comment

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

LGTM, locally confirmed that the path is correct again.

@wilfred-s wilfred-s closed this in 0a497a6 Jan 29, 2026
wilfred-s pushed a commit that referenced this pull request Jan 29, 2026
After upgrading to Angular 20, the default output directory has a
"browser" prefix prepended. Remove this to ensure content matches
previously generated app.

Closes: #250

Signed-off-by: Wilfred Spiegelenburg <wilfreds@apache.org>
(cherry picked from commit 0a497a6)
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.

2 participants