Skip to content

Default to one experiment visible, enable multi-experiment choices through URL#228

Merged
Luke-Schreiber merged 7 commits intomainfrom
experiment-switching-url-param
Feb 27, 2026
Merged

Default to one experiment visible, enable multi-experiment choices through URL#228
Luke-Schreiber merged 7 commits intomainfrom
experiment-switching-url-param

Conversation

@Luke-Schreiber
Copy link

@Luke-Schreiber Luke-Schreiber commented Feb 3, 2026

Does this PR close any open issues?

Closes #226

Give a longer description of what this PR addresses and why it's needed

  • Flag to enable experiment visibility control: --experiment-visibility-control flag allows for users to set certain experiments as visible in aa_index.json.
  • If --experiment-visibility-control flag set, on http://localhost:5173, only experiments defined as visible in aa_index.json show.
  • On http://localhost:5173/?show-all-experiments, all experiments become visible.

@Luke-Schreiber Luke-Schreiber self-assigned this Feb 3, 2026
@Luke-Schreiber Luke-Schreiber added the frontend Deals with Vue Frontend label Feb 3, 2026
@Luke-Schreiber Luke-Schreiber marked this pull request as ready for review February 3, 2026 21:37
@Luke-Schreiber
Copy link
Author

  • Base URL - no experiments available.
  • Run this by Shukran & Tom one more time.

Comment on lines +151 to +152
'show-experiments' in query ||
currentFilename == null
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should consider if the base url should show no experiments.

Copy link

@Dev-Lan Dev-Lan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just the one comment about default behavior with no url params.

@Luke-Schreiber
Copy link
Author

Luke-Schreiber commented Feb 3, 2026

  • Make this hiding experiments functionality a build flag.

Copy link

@Dev-Lan Dev-Lan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good 👍

@Luke-Schreiber Luke-Schreiber merged commit 10f17b2 into main Feb 27, 2026
@Luke-Schreiber Luke-Schreiber deleted the experiment-switching-url-param branch February 27, 2026 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend Deals with Vue Frontend

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Sharing single experiments (with other experiments hidden)

2 participants