Skip to content

fix: apply trackio fixes for experiment ID and modal behavior#1684

Merged
dadmobile merged 1 commit intotransformerlab:mainfrom
orincolor:fix-trackio
Mar 29, 2026
Merged

fix: apply trackio fixes for experiment ID and modal behavior#1684
dadmobile merged 1 commit intotransformerlab:mainfrom
orincolor:fix-trackio

Conversation

@orincolor
Copy link
Copy Markdown

Screenshot 2026-03-28 at 10 07 04 PM

The endpoint was reading experiment_id from auth context (where it was
never set), always getting None and raising a 400 error.

  • Backend: add experiment_id as an explicit query param to /trackio/start
  • Frontend: pass experimentId prop to TrackioModal and include it in the
    fetch URL; also replace localhost/127.0.0.1 in the returned Trackio URL
    with the actual server hostname so the iframe loads correctly in remote
    browsers

@sentry
Copy link
Copy Markdown

sentry bot commented Mar 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@dadmobile dadmobile merged commit 1a26ab2 into transformerlab:main Mar 29, 2026
11 of 13 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.

3 participants