-
Notifications
You must be signed in to change notification settings - Fork 467
feat: renamed-feature-health-sample-to-webhook ⚠️ #6332
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 3 Skipped Deployments
|
for more information, see https://pre-commit.ci
Docker builds report
|
api/features/feature_health/migrations/0003_migrate_sample_to_generic.py
Outdated
Show resolved
Hide resolved
…/flagsmith into feat/migrate-sample-to-generic
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.
This is the final PR Bugbot will review for you during this billing cycle
Your free Bugbot reviews will reset on December 10
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
api/features/feature_health/migrations/0003_migrate_sample_to_webhook.py
Show resolved
Hide resolved
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6332 +/- ##
=======================================
Coverage 98.02% 98.02%
=======================================
Files 1280 1281 +1
Lines 45406 45461 +55
=======================================
+ Hits 44508 44563 +55
Misses 898 898 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
khvn26
left a comment
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.
LGTM 👍
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature!Contributes to #6130
Changes
In this PR, we migrate the
SampleFeature Health Provider toWebhook:How did you test this code?
Edit to webhook - same records
