Skip to content

New: [AEA-5696] - Add feedback into EPS Assist Me#33

Closed
kris-szlapa wants to merge 45 commits intomainfrom
AEA-5696-add-feedback
Closed

New: [AEA-5696] - Add feedback into EPS Assist Me#33
kris-szlapa wants to merge 45 commits intomainfrom
AEA-5696-add-feedback

Conversation

@kris-szlapa
Copy link

Summary

🎫 AEA-5696 Add feedback into EPS Assist Me

✨ New Feature

Details

Add a feedback mechanism to EPS Assist Me so that after a supplier receives an answer, they are asked “Was this helpful?”. If they select Yes, the response is stored and a thank-you message is shown, and if they select No, they are prompted to provide written feedback.

Copy link
Contributor

@bencegadanyi1-nhs bencegadanyi1-nhs left a comment

Choose a reason for hiding this comment

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

Added some comments, looks good, gonna hold off approving just for now until our group discussion later

Copy link
Contributor

@bencegadanyi1-nhs bencegadanyi1-nhs left a comment

Choose a reason for hiding this comment

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

LGTM overall, I’ve just added a couple of non-blocking comments - let’s add a bit more test coverage before merging but it’s getting there 👍

@sonarqubecloud
Copy link

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.

3 participants