Skip to content

Conversation

@zhiyuanliang-ms
Copy link
Member

@zhiyuanliang-ms zhiyuanliang-ms commented Jan 24, 2024

@zhiyuanliang-ms zhiyuanliang-ms marked this pull request as ready for review January 25, 2024 01:52
// Microsoft feature flag schema: https://github.com/Azure/AppConfiguration/blob/main/docs/FeatureManagement/FeatureFlag.v2.0.0.schema.json
internal static class MicrosoftFeatureFlagFields
{
public const string FeatureFlagsSectionName = "FeatureFlags";
Copy link
Member

@rossgrambo rossgrambo Feb 16, 2024

Choose a reason for hiding this comment

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

Lets add support for "feature_flags" & "feature_management" as well here.

Copy link
Member Author

Choose a reason for hiding this comment

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

I think we should only support snake casing for Microsoft Feature Management schema.
@jimmyca15

Copy link
Member

@rossgrambo rossgrambo Feb 20, 2024

Choose a reason for hiding this comment

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

The rest of this file is snake case, but perhaps I'm misunderstanding how this file is used?

*I now see the updated PR

@zhiyuanliang-ms zhiyuanliang-ms changed the title Support Microsoft Feature Flag schema v2.0.0 for preview branch Support Microsoft Feature Management schema for preview branch Feb 18, 2024
@zhiyuanliang-ms zhiyuanliang-ms deleted the zhiyuanliang/support-microsoft-schema-for-preview branch February 26, 2024 03:07
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.

5 participants