Skip to content

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

Merged
Luke-Schreiber merged 7 commits into
mainfrom
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 into
mainfrom
experiment-switching-url-param

Conversation

@Luke-Schreiber

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

Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown

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.

@Dev-Lan Dev-Lan left a comment

Copy link
Copy Markdown

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

Luke-Schreiber commented Feb 3, 2026

Copy link
Copy Markdown
Author
  • Make this hiding experiments functionality a build flag.

@Dev-Lan Dev-Lan left a comment

Copy link
Copy Markdown

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