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.

Sign up for GitHub

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

Jump to bottom

DTSCCI-136 use openid connect to use existing session to connect. #3679

Closed
wants to merge 35 commits into from
Closed

DTSCCI-136 use openid connect to use existing session to connect. #3679

wants to merge 35 commits into from

Conversation

velimd
Copy link
Contributor

@velimd velimd commented Aug 5, 2024

JIRA link

https://tools.hmcts.net/jira/browse/DTSCCI-136

Change description

Work checklist

  • Unit tests added where applicable
  • Route tests added for new pages
  • New pages included in a11y tests
  • Task list and task completeness checks updated
  • Check and send page updated
  • Routes, page content and page flows of new features are toggled
  • UI changes look good on mobile
  • Required Google Analytics events are being sent

Developer self-QA run statement

  • I have clicked through the running application to see if all changes I made actually work.

Does this PR introduce a breaking change?

  • Yes
  • No

@velimd velimd requested a review from a team as a code owner August 5, 2024 16:05
@velimd velimd requested a review from nigeldunne August 12, 2024 13:34
ruban72
ruban72 previously approved these changes Aug 12, 2024
View reviewed changes
ed14537
ed14537 previously approved these changes Aug 13, 2024
View reviewed changes
@velimd velimd requested review from ruban72 and ed14537 August 16, 2024 15:24
rishikrsharma
rishikrsharma previously approved these changes Aug 21, 2024
View reviewed changes
nigeldunne
nigeldunne previously approved these changes Aug 23, 2024
View reviewed changes
@velimd velimd closed this Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rishikrsharma rishikrsharma rishikrsharma left review comments

@nigeldunne nigeldunne nigeldunne left review comments

@ruban72 ruban72 Awaiting requested review from ruban72

@ed14537 ed14537 Awaiting requested review from ed14537

Assignees
No one assigned
Labels
enable_keep_helm ns:money-claims prd:cmc rel:cmc-citizen-frontend-pr-3679
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@velimd @rishikrsharma @nigeldunne @ruban72 @ed14537