Skip to content
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

Merged
merged 20 commits into from
Apr 21, 2023
Merged

Remove Launchdarkly #4681

merged 20 commits into from
Apr 21, 2023

Conversation

thetif
Copy link
Contributor

@thetif thetif commented Apr 17, 2023

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

  1. Login as betauser and see that New Apdland is not available
  2. Login as norole and see that New Apdland is available
  3. Login as sysadmin and see that you have access to all states, click on one state
  4. Verify that the Create new APD button isn't visible
  5. Login as state admin, verify that you can create an MMIS APD and that it looks right

This pull request is ready to code review when

  • Automated tests are updated (and all tests are passing)
  • Chromatic link added above
  • Pull request has been labeled, if applicable with feature, content, bug,
    tests, refactor

This pull request is ready to test when

  • Code has been reviewed by someone other than the original author

This pull request is ready to review when QA has

  • Verified the functionality related to the change
  • Verified that the change works with Narrator on Windows
  • Verified that the change works with VoiceOver on Mac
  • Verified all updated pages with the WAVE tool
  • Verified tab and keyboard navigation functionality

This pull request can be merged when

  • Design has approved the experience
  • Product has approved the experience

@codecov
Copy link

codecov bot commented Apr 17, 2023

Codecov Report

Merging #4681 (0a55127) into main (5639977) will decrease coverage by 1.14%.
The diff coverage is 85.71%.

Impacted file tree graph

@@            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     
Flag Coverage Δ
api ∅ <ø> (∅)
common 99.17% <100.00%> (+0.42%) ⬆️
web 90.17% <76.92%> (-1.23%) ⬇️
Impacted Files Coverage Δ
e2e/cypress/helpers/apd/apd-overview.js 82.45% <ø> (-0.31%) ⬇️
...e/cypress/helpers/apd/assurances-and-compliance.js 60.58% <ø> (-0.41%) ⬇️
...press/helpers/apd/date-dollar-inline-validation.js 100.00% <ø> (ø)
e2e/cypress/helpers/apd/proposed-budget.js 61.76% <ø> (+0.33%) ⬆️
e2e/cypress/support/commands.js 67.45% <ø> (-5.94%) ⬇️
web/src/containers/ApdApplication.js 96.96% <ø> (-3.04%) ⬇️
web/src/containers/App.js 100.00% <ø> (ø)
web/src/containers/Root.js 100.00% <ø> (ø)
web/src/pages/apd/export/ApdExport.js 100.00% <ø> (+6.66%) ⬆️
web/src/util/hooks.js 0.00% <0.00%> (-100.00%) ⬇️
... and 6 more

... and 18 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5639977...0a55127. Read the comment docs.

@cms-eapd-bot
Copy link

cms-eapd-bot commented Apr 19, 2023

This deploy was cleaned up.

@thetif thetif added the refactor Issues which require varying degrees of code refactoring label Apr 20, 2023
@thetif thetif marked this pull request as ready for review April 20, 2023 14:03
@thetif thetif requested review from Sun-Mountain, tbolt and amyd11 April 20, 2023 14:04
@thetif thetif changed the base branch from main to tforkner/4056-update-okta-connection April 20, 2023 16:54
Base automatically changed from tforkner/4056-update-okta-connection to main April 20, 2023 17:55
Copy link
Contributor

@tbolt tbolt left a 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 />
Copy link
Contributor

Choose a reason for hiding this comment

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

😄

Copy link
Contributor

@amyd11 amyd11 left a 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

@thetif thetif requested review from mirano-darren and removed request for Sun-Mountain April 20, 2023 19:50
Copy link
Contributor

@mirano-darren mirano-darren left a 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

@thetif thetif merged commit 27e40b1 into main Apr 21, 2023
@thetif thetif deleted the tforkner/remove-launchdarkly branch April 21, 2023 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Issues which require varying degrees of code refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants