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-3628 Request a key templates to work with combined format #2978

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

Conversation

SeemaMenon
Copy link
Contributor

Purpose

Refactoring request a key templates to work with combined LPA format

Fixes UML-3628

Approach

Explain how your code addresses the purpose of the change

Learning

Any tips and tricks, blog posts or tools which helped you. Plus anything notable you've discovered about the Use a Lasting Power of Attorney service

Checklist

  • I have performed a self-review of my own code
  • I have added relevant logging with appropriate levels to my code
  • New event_codes have been documented on the wiki page
  • I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
  • I have added tests to prove my work
  • I have added welsh translation tags and updated translation files
  • I have run an accessibility tool on any pages I have made changes to and fixed any issues found
  • I have notified the Interaction Designer of any content changes so that appropriate screenshots/flow diagram changes can be made
  • The product team have tested these changes

@SeemaMenon SeemaMenon requested a review from a team as a code owner December 3, 2024 23:16
@github-actions github-actions bot added php Pull requests that update Php code service-front labels Dec 3, 2024
Copy link

codecov bot commented Dec 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.95%. Comparing base (818972a) to head (a845d5b).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main    #2978   +/-   ##
=========================================
  Coverage     91.95%   91.95%           
  Complexity     1781     1781           
=========================================
  Files           390      390           
  Lines          8275     8276    +1     
=========================================
+ Hits           7609     7610    +1     
  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.70% <100.00%> (+<0.01%) ⬆️
Files with missing lines Coverage Δ
...tor/src/Handler/Factory/CheckLpaHandlerFactory.php 100.00% <100.00%> (ø)

@SeemaMenon SeemaMenon requested a review from allenannom December 4, 2024 00:45
@SeemaMenon SeemaMenon force-pushed the UML-3628-template-refactor branch from b702290 to 59cddd0 Compare December 9, 2024 13:04
@github-actions github-actions bot added terraform Pull requests that update Terraform code docker Pull requests that update Docker code labels Dec 9, 2024
@@ -109,7 +109,7 @@ services:
ALLOW_MERIS_LPAS: "false"
INSTRUCTIONS_AND_PREFERENCES: "true"
ALLOW_GOV_ONE_LOGIN: "true"
SUPPORT_DATASTORE_LPAS: "false"
SUPPORT_DATASTORE_LPAS: "true"
Copy link
Contributor

Choose a reason for hiding this comment

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

this should be set to false as the feature isn't ready for production yet

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, I need a run to get the dev links for UAT. will revert after UAT

@@ -62,7 +62,7 @@
"allow_meris_lpas": false,
"instructions_and_preferences": true,
"allow_gov_one_login": true,
"support_datastore_lpas": false
"support_datastore_lpas": true
Copy link
Contributor

Choose a reason for hiding this comment

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

flag should be turned off here too

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 php Pull requests that update Php code service-front terraform Pull requests that update Terraform code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants