-
Notifications
You must be signed in to change notification settings - Fork 30
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
Remove Launchdarkly #4681
Remove Launchdarkly #4681
Conversation
…ience; updated the set-up-users to use the new users that were created a while ago
Codecov Report
@@ Coverage Diff @@
## main #4681 +/- ##
==========================================
- Coverage 91.78% 90.65% -1.14%
==========================================
Files 288 288
Lines 9300 9252 -48
Branches 1891 1878 -13
==========================================
- Hits 8536 8387 -149
- Misses 735 826 +91
- Partials 29 39 +10
... and 18 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
This deploy was cleaned up. |
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.
Looks good
@@ -94,7 +88,7 @@ const ApdApplication = ({ | |||
<ApdPageRoutes apdId={paramApdId} /> | |||
</div> | |||
</main> | |||
{adminCheckFlag === true && <AdminCheckPanel />} | |||
<AdminCheckPanel /> |
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.
😄
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 taking all this out
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.
Looks good to me! All accounts behave as expected
Resolves the need for LaunchDarkly
Description
Removes LaunchDarkly from the application. Turns all previously flagged code "on". Fully enables MMIS and adminCheck. Created a work around for the new code that shows or hides New Apdland. Created work around for sysadmin and just hardcoded access for the username 'sysadmin'.
Chromatic Link
https://www.chromatic.com/pullrequest?appId=61d5b948cf6f17003a12bf77&number=4681
Significant changes or possible side effects
Enabled all beta feature code.
Steps to manually verify this change
This pull request is ready to code review when
tests, refactor
This pull request is ready to test when
This pull request is ready to review when QA has
This pull request can be merged when