-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[AppConfig] appconfig feature list/show and appconfig kv import/export
: Support microsoft feature management schema
#30376
Merged
yanzhudd
merged 49 commits into
Azure:dev
from
ChristineWanjau:cwanjau/supportFFV2Schema
Jan 22, 2025
Merged
Changes from 47 commits
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
a242281
Support new ms fm schema
bc99149
update tests
9d077e9
update imports
52c2d31
update flag properties
90d876a
Merge branch 'cwanjau/supportFFV2Schema' of https://github.com/Christ…
c846079
update feature properties
9396e6d
update feature models
60b9d0f
update test
ec96b1a
update test files
ab36eeb
update feature models
e5fbdc3
update feature models
9ec9e81
update feature models
2dd09da
update tests
e9dc52c
update filter
1d0565b
update boolean value
1e1c580
update tests
359e19e
Merge branch 'cwanjau/supportFFV2Schema' of https://github.com/Christ…
b6e4060
update tests
62d2157
fix style issues
ddfbfcb
update default compatibility mode
bd57035
Push test recordings
60f087c
Revert "Push test recordings"
5203b67
resolve merge conflicts
2db2b9c
push test recordings
e31f034
resolve test failures
a996a91
add cli tests
0fba17f
add invalid variants test
93a985f
add tests
8fb54d3
address comments
a3dc72b
resolve styling issue
46f4f21
Address comments
1a4e73e
Merge branch 'cwanjau/supportFFV2Schema' of https://github.com/Christ…
4719333
renamed variable
9f2686b
address class comments
6af239a
update setting feature as keyvalue
d7cd6e2
push test recordings
cded062
address comments
4765742
resolve merge conflicts
e33ba49
resolve comments
bf0c7be
address comments
3a0b2ba
update according to comments
b683b90
fix styling issues
8972437
update tests
bc6bdbc
address comments
d7f33f5
code improvements
550aabd
update spelling
03ce314
push tests
3c01345
Update export to file help message
1b294a1
Update export to file help message
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When we display enabled property instead of state, we must also display the number of filters/variants and if telemetry is enabled.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes we currently display variants/filters and telemetry for both scenarios with state property and enabled property.