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

Uml 3764 remove one login feature flag #3032

Open
wants to merge 26 commits into
base: main
Choose a base branch
from

Conversation

allenannom
Copy link
Contributor

Purpose

removing the gov one login feature flag and usages

Fixes UML-3764

Checklist (tick/delete or strikethrough as appropriate)

  • I have performed a self-review of my own code

@allenannom allenannom requested a review from a team as a code owner January 6, 2025 18:08
# Conflicts:
#	terraform/environment/region/api_ecs.tf
#	terraform/environment/terraform.tfvars.json
@github-actions github-actions bot added php Pull requests that update Php code terraform Pull requests that update Terraform code docker Pull requests that update Docker code labels Jan 6, 2025
Copy link

codecov bot commented Jan 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.94%. Comparing base (7afefc6) to head (bc8a154).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #3032      +/-   ##
============================================
- Coverage     91.95%   91.94%   -0.01%     
+ Complexity     1781     1779       -2     
============================================
  Files           390      390              
  Lines          8276     8268       -8     
============================================
- Hits           7610     7602       -8     
  Misses          649      649              
  Partials         17       17              
Flag Coverage Δ
use-an-lpa-admin 78.85% <ø> (ø)
use-an-lpa-api 97.31% <ø> (ø)
use-an-lpa-front 90.68% <100.00%> (-0.02%) ⬇️
Files with missing lines Coverage Δ
...r/src/Handler/Factory/LogoutPageHandlerFactory.php 100.00% <100.00%> (ø)
...Authentication/AuthenticationMiddlewareFactory.php 100.00% <ø> (ø)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docker Pull requests that update Docker code github-actions php Pull requests that update Php code service-api service-front terraform Pull requests that update Terraform code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant