-
Notifications
You must be signed in to change notification settings - Fork 467
fix(LD Importer): Archive deprecated LaunchDarkly flags on import #6447
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
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 3 Skipped Deployments
|
Docker builds report
|
docs/docs/administration-and-security/data-management/import-from-launchdarkly.md
Outdated
Show resolved
Hide resolved
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6447 +/- ##
=======================================
Coverage 98.07% 98.07%
=======================================
Files 1294 1294
Lines 46535 46545 +10
=======================================
+ Hits 45637 45647 +10
Misses 898 898 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
601ab7f to
fdcec4b
Compare
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!
fdcec4b to
cff70ec
Compare
…ve-deprecated-flags # Conflicts: # frontend/web/components/pages/FeaturesPage.js
|
You have run out of free Bugbot PR reviews for this billing cycle. This will reset on January 10. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
task_processor is now part of flagsmith-common and doesn't need to be cloned separately. This has been broken since August 2024 when the clone was added without the corresponding pyproject.toml entry.
Deprecated LaunchDarkly flags are imported as archived flags with a "Deprecated" tag to reduce confusion during the import process.
Changes
Closes #5642
Review effort: 2/5
Warning
My frontend mastery is very out of date. UI changes in this patch rely on interactions with AI.