Skip to content

Conversation

@mfcrocker
Copy link
Contributor

What and why?

Improvements to the edit account details journey

How to test?

Go to my account and edit account details, check it matches the Figma journey (I've cleared with Alina that we should be maintaining the functionality around sending a confirmation email, but have tidied it up significantly to fit with the new design)

Jira

RAS-1449

@mfcrocker mfcrocker requested a review from a team as a code owner February 26, 2025 14:00
@mfcrocker
Copy link
Contributor Author

/deploy crockers

@ras-rm-pr-bot
Copy link
Collaborator

Deploying to dev cluster with following parameters:

  • namespace: crockers

  • tag: pr-1044

  • configBranch: main

  • paramKey: ``

  • paramValue: ``

@mfcrocker
Copy link
Contributor Author

/deploy crockers

@ras-rm-pr-bot
Copy link
Collaborator

Deploying to dev cluster with following parameters:

  • namespace: crockers

  • tag: pr-1044

  • configBranch: main

  • paramKey: ``

  • paramValue: ``

@mfcrocker
Copy link
Contributor Author

/deploy crockers

@ras-rm-pr-bot
Copy link
Collaborator

Deploying to dev cluster with following parameters:

  • namespace: crockers

  • tag: pr-1044

  • configBranch: main

  • paramKey: ``

  • paramValue: ``

@mfcrocker
Copy link
Contributor Author

/deploy crockers

@ras-rm-pr-bot
Copy link
Collaborator

Deploying to dev cluster with following parameters:

  • namespace: crockers

  • tag: pr-1044

  • configBranch: main

  • paramKey: ``

  • paramValue: ``

Copy link
Contributor

@LJBabbage LJBabbage left a comment

Choose a reason for hiding this comment

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

Looks like a problem when it errors. I just pressed change details, and got this. I got the same when I forced the email to error, i.e removed the @

Screenshot 2025-03-04 at 13 41 38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants