Skip to content

Change serving hardcoded default to the variation name - #5

Merged
leslie-lau merged 2 commits into
mainfrom
change-serving-variation-name-from-default
Aug 25, 2026
Merged

Change serving hardcoded default to the variation name#5
leslie-lau merged 2 commits into
mainfrom
change-serving-variation-name-from-default

Conversation

@leslie-lau

Copy link
Copy Markdown

The Variation Name was hardcoded to "Default". Changed it to the actual Variation that the user is bucketed into.

Screenshot 2026-08-25 at 3 04 39 PM

Copilot AI 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.

Pull request overview

This PR updates the UI to display the actual DevCycle/OpenFeature variation name a user is bucketed into (instead of the previously hardcoded "Default"), and adjusts provider initialization so the DevCycle provider is created once rather than on each App render.

Changes:

  • Read the active variation name from the DevCycle client’s feature data for hello-togglebot and render it in ToggleBot.
  • Initialize DevCycleReactProvider and set the OpenFeature context/provider at module scope in App.tsx.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
src/components/ToggleBot.tsx Pull variation name from the DevCycle client via the OpenFeature provider and render it in the UI.
src/App.tsx Move DevCycle provider creation and OpenFeature setContext/setProvider out of the App component to run once.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/components/ToggleBot.tsx Outdated
Comment thread src/App.tsx
Comment thread src/components/ToggleBot.tsx Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Comment thread src/App.tsx
@leslie-lau
leslie-lau merged commit bfb267d into main Aug 25, 2026
4 checks passed
@leslie-lau
leslie-lau deleted the change-serving-variation-name-from-default branch August 25, 2026 19:33
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.

4 participants