-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
feat(flags): Add C# language examples for feature flags #28702
Conversation
Note that this doesn't include the recently shipped remote config setting code. I'll get to that soon. |
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.
PR Summary
Added C# language support for feature flags in PostHog, including code snippets, documentation links, and UI elements for the .NET SDK.
- Added
CSharpSnippet
component in/frontend/src/scenes/feature-flags/FeatureFlagSnippets.tsx
with async/await patterns and C# naming conventions - Added
DOTNET
SDK key toSDKKey
enum in/frontend/src/types.ts
and included in local evaluation/payload libraries lists - Added
IconCSharp
component in/frontend/src/lib/lemon-ui/icons/icons.tsx
for C# language representation - Added C# language support to
CodeSnippet
component with Prism syntax highlighting integration - Added C# implementation examples with proper documentation links in
FeatureFlagCodeOptions.tsx
5 file(s) reviewed, 3 comment(s)
Edit PR Review Bot Settings | Greptile
Size Change: +5 B (0%) Total Size: 1.21 MB ℹ️ View Unchanged
|
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
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.
nice work nice work!
I need #28705 to land before I can merge this. |
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Problem
Adds C# as a language in the "How to implement" section when looking at the details for a feature flag.
Changes
Some examples.
Does this work well for both Cloud and self-hosted?
Yes.
How did you test this code?
Manually tested.