Skip to content

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

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
Merged
anthony-nhs merged 73 commits into from
Sep 17, 2025
Merged

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

anthony-nhs merged 73 commits into from
Sep 17, 2025

Conversation

@anthony-nhs
Copy link
Contributor

@anthony-nhs anthony-nhs commented Sep 16, 2025 •
edited
Loading

Summary

  • Routine Change

Details

  • add feedback
  • use layers for python dependencies
  • add working cdk-watch makefile target

@kris-szlapa
Add feedback functionality with interactive buttons and feedback storage
7a6d4e0
@kris-szlapa
Move imports to top of the file
d3a9e4b
@kris-szlapa
Add comments to feedback handlers
0ffe006
@kris-szlapa
Move feedback messange above buttons
75aa4f1
@kris-szlapa
Add feedback test suite
9b6e659
@kris-szlapa
Update test with feedback blocks
b0efa00
@kris-szlapa
Add slack handlers test suite
71773a5
@kris-szlapa
Add feedback handling to @mentions and fix feedback message processing
22c1d16
@kris-szlapa
Remove app middleware from slack handler
a2d51ee
@kris-szlapa
Update feedback message
c1f0df2
@kris-szlapa
Merge branch 'main' of https://github.com/NHSDigital/eps-assist-me into
236cb99 
AEA-5696-add-feedback
@kris-szlapa
Add feedback button handlers and unify message handler
6544f9e
@kris-szlapa
Update comments
373ccd0
@kris-szlapa
Optimise function to handle whitespace by stripping once after regex …
7a0b3b8 
…replacement
@kris-szlapa
Add button text to BOT_MESSAGES dictionary
7cb5c4c
@kris-szlapa
Replace hardcoded strings with constants across codebase
f46b0c7
@kris-szlapa
Add TTL constants to config file
1788e7d
@kris-szlapa
Combine feedback handlers into a single unified handler
003e0d8
@kris-szlapa
Merge branch 'main' of https://github.com/NHSDigital/eps-assist-me into
2243a57 
AEA-5696-add-feedback
@kris-szlapa
Update feedback pattern
23b7755
@kris-szlapa
Update function names in tests
5e24f82
@kris-szlapa
Ignore inactive button clicks and store feedback with latest_message_ts
777f204
@kris-szlapa
Update test suits
d39b31f
@kris-szlapa
Remove log_event true from inject_lambda_context
0e2ba42
@kris-szlapa
Add logic to clean up previous unfeedback Q&A before storing new one
f6135c3
@kris-szlapa
Update test suites to improve coverage
e7ceee0
@kris-szlapa
Add more tests to suits
45b2c7a
@kris-szlapa
Add tests for feedback logic
e8e3298
@kris-szlapa
Remove the unused function parameter body
47c7690
@kris-szlapa
Refactor store_feedback_with_qa function to reduce its cognitive comp…
8018e71 
…lexity
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 17, 2025

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
84.8% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@anthony-nhs anthony-nhs merged commit 9113b02 into main Sep 17, 2025
9 checks passed
@anthony-nhs anthony-nhs deleted the new_feedback_branch branch September 17, 2025 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@anthony-nhs @kris-szlapa