Skip to content

Add deep linking for canvas extensions#2256

Closed
jamesmontemagno wants to merge 1 commit into
github:mainfrom
jamesmontemagno:motz-deep-link-extensions
Closed

Add deep linking for canvas extensions#2256
jamesmontemagno wants to merge 1 commit into
github:mainfrom
jamesmontemagno:motz-deep-link-extensions

Conversation

@jamesmontemagno

Copy link
Copy Markdown
Contributor

Summary

  • Added deep-link support for Canvas Extensions using ?extension=<id>.
  • On page load, the matching extension modal opens automatically when the query param is valid.
  • If the query param references an unknown extension, it is removed from the URL.

URL state behavior

  • Opening an extension now writes extension=<id> to the URL.
  • Closing the modal removes the extension query param so shared links stay clean and accurate.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 9, 2026 19:59
@github-actions github-actions Bot added the website-update PR touches website content or code label Jul 9, 2026
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

🔴 Contributor Reputation Check: HIGH risk

Check Risk
Profile HIGH
Credential audit NONE

Maintainers: please review this contributor before merging.
See the workflow run for full details.
Automated check powered by AGT.

@github-actions github-actions Bot added the needs-review:HIGH Contributor reputation check flagged HIGH risk label Jul 9, 2026

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

This PR adds URL-based deep linking for Canvas Extension detail modals on the extensions page by syncing the currently open extension to an extension=<id> query parameter. It also auto-opens the referenced extension modal on page load and cleans up the URL when the modal is closed or when an invalid extension ID is provided.

Changes:

  • Introduced extension query parameter sync when opening/closing an extension details modal.
  • Added page-load behavior to auto-open the extension modal when ?extension=<id> is present and valid.
  • Removed invalid extension query parameters from the URL when the ID is unknown.
Show a summary per file
File Description
website/src/scripts/pages/extensions.ts Adds deep-link query param synchronization, initial modal open from URL, and a modal close observer to remove the param.

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Low

@aaronpowell

Copy link
Copy Markdown
Contributor

We're overhauling these pages with #2246 which will give them all dedicated URLs, so I'm going to close this

@aaronpowell aaronpowell closed this Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review:HIGH Contributor reputation check flagged HIGH risk website-update PR touches website content or code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants