Skip to content

fix(*): platform switcher correct url so that no client-side redirect…#436

Merged
dobromirts merged 3 commits into
vnextfrom
sstoychev/platform-switch-direct-url
Jul 17, 2026
Merged

fix(*): platform switcher correct url so that no client-side redirect…#436
dobromirts merged 3 commits into
vnextfrom
sstoychev/platform-switch-direct-url

Conversation

@ChronosSF

Copy link
Copy Markdown
Member

Uses full urls to the getting started page instead of relying on ugly client-side redirect.

@ChronosSF
ChronosSF requested a review from dobromirts July 15, 2026 15:22
@dobromirts dobromirts added the 💥 status: in-test PRs currently being tested label Jul 16, 2026
@dobromirts
dobromirts requested a review from Copilot July 16, 2026 07:46

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.

Pull request overview

Updates the docs platform switcher links to point directly to each platform’s “Getting Started” page (avoiding a client-side redirect) across the Angular and xplat Astro site configurations.

Changes:

  • Updated per-platform URL metadata in IGDOCS_PLATFORMS.
  • Adjusted platform switcher link construction to stop appending a trailing /.
  • Updated both docs/angular and docs/xplat configs to use the revised URLs.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
src/platform.ts Modified platform URL metadata used by Astro site site/base config and switcher link generation.
docs/xplat/astro.config.ts Updated platform switcher href generation (production/staging).
docs/angular/astro.config.ts Updated platform switcher href generation (production/staging).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/platform.ts
Comment thread src/platform.ts
Comment thread docs/angular/astro.config.ts
Comment thread docs/xplat/astro.config.ts
Comment thread src/platform.ts Outdated
@dobromirts

Copy link
Copy Markdown
Contributor

Without the copilot suggesting locally after building angular stating and starting it with the preview command based on the dist input all links are with incorrect state (included general path for each) -
image

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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

@ChronosSF
ChronosSF requested a review from dobromirts July 17, 2026 08:13
@dobromirts dobromirts added ✅ status: verified Applies to PRs that have passed manual verification and removed 💥 status: in-test PRs currently being tested labels Jul 17, 2026
@dobromirts
dobromirts merged commit e4c74ae into vnext Jul 17, 2026
7 checks passed
@dobromirts
dobromirts deleted the sstoychev/platform-switch-direct-url branch July 17, 2026 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants