Skip to content

Fix: Open browser with write_token so trackio show allows mutations#509

Merged
abidlabs merged 3 commits into
mainfrom
fix-issue-497-write-token-url
Apr 17, 2026
Merged

Fix: Open browser with write_token so trackio show allows mutations#509
abidlabs merged 3 commits into
mainfrom
fix-issue-497-write-token-url

Conversation

@abidlabs
Copy link
Copy Markdown
Member

@abidlabs abidlabs commented Apr 17, 2026

Summary

Closes #497.

trackio show always opens the dashboard in read-only mode because the browser is opened with dashboard_url (no write_token) instead of full_url (includes write_token). Without the token, the trackio_write_token cookie is never set, so get_run_mutation_status returns allowed=false and the UI disables Rename/Delete.

This PR switches the browser open and the notebook embed to use full_url, matching the existing docstring that describes full_url as "the full URL including the write token". The print(...) line still shows the tokenless dashboard_url so the token is not leaked to terminal logs.

Changes

  • trackio/__init__.py: use full_url in webbrowser.open(...) and utils.embed_url_in_notebook(...).

Started with CC, tested and updated manually

Closes #497

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@gradio-pr-bot
Copy link
Copy Markdown
Contributor

gradio-pr-bot commented Apr 17, 2026

🪼 branch checks and previews

Name Status URL
🦄 Changes detected! Details

@gradio-pr-bot
Copy link
Copy Markdown
Contributor

🦄 change detected

This Pull Request includes changes to the following packages.

Package Version
trackio minor

  • Fix: Open browser with write_token so trackio show allows mutations

‼️ Changeset not approved. Ensure the version bump is appropriate for all packages before approving.

  • Maintainers can approve the changeset by checking this checkbox.

Something isn't right?

  • Maintainers can change the version label to modify the version bump.
  • If the bot has failed to detect any changes, or if this pull request needs to update multiple packages to different versions or requires a more comprehensive changelog entry, maintainers can update the changelog file directly.

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

HuggingFaceDocBuilderDev commented Apr 17, 2026

🪼 branch checks and previews

Name Status URL
Spaces ready! Spaces preview

Install Trackio from this PR (includes built frontend)

pip install "https://huggingface.co/buckets/trackio/trackio-wheels/resolve/f7b6203050b1bb690e3aa44ebc1e20b39b180bf3/trackio-0.23.0-py3-none-any.whl"

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@abidlabs abidlabs marked this pull request as ready for review April 17, 2026 18:15
@abidlabs abidlabs merged commit 21c099a into main Apr 17, 2026
8 of 9 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.

Cannot rename or delete runs

3 participants