fix: point the extension's dashboard links at the current path-scoped URLs#617
Open
JulianMaurin wants to merge 1 commit into
Conversation
… URLs The dashboard moved to /orgs/<org>/repos/<repo>/… routing; the retired ?login=&repository= query form no longer resolves, so the extension's activity-log and merge-queue links (batch and non-batch) were landing on a dead route. Build them through a shared getDashboardRepoBase() helper; the query filters (pull_request, batch_pull, batch_pr, branch, pull-request-number) are unchanged. Test plan: - npx jest (72 passing) — every dashboard link asserts the path-scoped URL. - npx biome check — clean. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KJbLFjMX9mb8o5T3JXTAXL Change-Id: I3ea0d77ec82d5171847f15cba0e3d70356879454
Contributor
Author
|
This pull request is part of a Mergify stack:
|
Contributor
Merge Protections🔴 2 of 6 protections blocking · waiting on 👀 reviews
🔴 👀 Review RequirementsWaiting for
This rule is failing.
🔴 🔎 ReviewsWaiting for
This rule is failing.
Show 4 satisfied protections🟢 🤖 Continuous Integration
🟢 Enforce conventional commitMake sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 📕 PR description
🟢 🚦 Auto-queueWhen all merge protections are satisfied, this pull request will be queued automatically. |
There was a problem hiding this comment.
Pull request overview
Updates the browser extension’s generated Mergify Dashboard links to use the newer path-scoped routing under /orgs/<org>/repos/<repo>/…, replacing the deprecated ?login=&repository= query-based URLs so links no longer land on dead routes.
Changes:
- Added a shared
getDashboardRepoBase()helper to centralize construction of the repo-scoped dashboard base URL. - Updated activity-log and merge-queue link builders to use the path-scoped base while preserving existing query filters.
- Updated Jest expectations to assert the new path-scoped URLs.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/queue.js | Introduces getDashboardRepoBase() and switches dashboard link generation to path-scoped URLs. |
| src/tests/queue.test.js | Updates assertions to match the new path-scoped dashboard URLs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
JulianMaurin
marked this pull request as ready for review
July 18, 2026 05:43
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The dashboard moved to /orgs//repos//… routing; the retired
?login=&repository= query form no longer resolves, so the extension's
activity-log and merge-queue links (batch and non-batch) were landing on a dead
route. Build them through a shared getDashboardRepoBase() helper; the query
filters (pull_request, batch_pull, batch_pr, branch, pull-request-number) are
unchanged.
Test plan:
Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01KJbLFjMX9mb8o5T3JXTAXL