Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add "Experimental Features" in settings #1707

Merged
merged 20 commits into from
Oct 19, 2023

Conversation

asklar
Copy link
Member

@asklar asklar commented Oct 16, 2023

Summary of the pull request

Adds a new setting for controlling the state of Experimental Features; this will allow us to ship MVPs and get feedback sooner.

image

image

References and relevant issues

Needed for #1049

Detailed description of the pull request / Additional comments

  • Experimental Features are defined in a new experiments array in navconfig.json.
  • Tools can declare an experimentId; the tool will only be available if the experiment is enabled.
  • Moved navconfig.json -> navconfig.jsonc so we can have comments in there now.

Validation steps performed

Locally validated

PR checklist

@asklar asklar marked this pull request as ready for review October 16, 2023 19:00
settings/DevHome.Settings/Strings/en-us/Resources.resw Outdated Show resolved Hide resolved
src/NavConfig.jsonc Outdated Show resolved Hide resolved
src/Services/PageService.cs Outdated Show resolved Hide resolved
tools/Experiments/src/DevHome.Experiments.csproj Outdated Show resolved Hide resolved
tools/Experiments/src/ExperimentalFeatures.md Outdated Show resolved Hide resolved
tools/Experiments/src/Strings/en-us/Resources.resw Outdated Show resolved Hide resolved
tools/Experiments/src/Views/TestExperimentPage.xaml Outdated Show resolved Hide resolved
asklar and others added 11 commits October 17, 2023 12:20
Co-authored-by: Kristen Schau <47155823+krschau@users.noreply.github.com>
Co-authored-by: Kristen Schau <47155823+krschau@users.noreply.github.com>
Co-authored-by: Kristen Schau <47155823+krschau@users.noreply.github.com>
Co-authored-by: Kristen Schau <47155823+krschau@users.noreply.github.com>
@cinnamon-msft
Copy link
Contributor

Strings LGTM :)

Copy link
Contributor

@bbonaby bbonaby left a comment

Choose a reason for hiding this comment

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

This looks good to me. @EricJohnson327 and @krschau are we all good to approve?

@EricJohnson327 EricJohnson327 merged commit 19ff23a into main Oct 19, 2023
4 checks passed
@asklar asklar deleted the user/asklar/experimentalFeatures branch October 20, 2023 00:30
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.

Add support for building / testing new features in Dev Home without impacting existing customers
5 participants