Skip to content

fix URLs for self-hosted AL-Go - #2388

Merged
Maria Zhelezova (mazhelez) merged 4 commits into
microsoft:mainfrom
Freddy-DK:deliverbug
Oct 6, 2026
Merged

Maria Zhelezova (mazhelez) merged 4 commits into
microsoft:mainfrom
Freddy-DK:deliverbug

Conversation

@FreundDK

@FreundDK Freddy (Bunker) (FreundDK) commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

❔What, Why & How

Fixes #2389

URLs in localdevenv and the settings schema are wrong when running a self-hosted AL-Go from a fork.

This fix is also included in PR #2309 (this is where the problem occured for partners using my fork)

@FreundDK
Freddy (Bunker) (FreundDK) requested a review from a team as a code owner October 2, 2026 22:04
Copilot AI balanced review requested due to automatic review settings October 2, 2026 22:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Copilot review overview

Review effort: Lite
Findings: 2 Medium severity

Open (2)
What changed in this PR

Fixes incorrect hardcoded Microsoft URLs when running a self-hosted AL-Go from a fork so that generated content (e.g., local dev env + settings schema) points to the correct repository.

Changes:

  • Add release note entry for Issue 2389.
  • Adjust deployment URL-rewrite logic to also run for non-microsoft owners (fork/self-host scenarios).
File Description
RELEASENOTES.md Documents the fix for fork/self-host URL generation issues (Issue 2389).
Internal/​Deploy.ps1 Expands URL replacement condition to cover forked/self-hosted repos, not just preview branch deployments.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread Internal/Deploy.ps1
Comment thread Internal/Deploy.ps1 Outdated
@mazhelez
Maria Zhelezova (mazhelez) merged commit cc7e9a9 into microsoft:main Oct 6, 2026
6 checks passed
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.

[Bug]: When deploying from an AL-Go fork, some URLs are wrong

5 participants