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

Add activity logging #1314

Merged
merged 16 commits into from
Jan 21, 2025
Merged

Add activity logging #1314

merged 16 commits into from
Jan 21, 2025

Conversation

Kizr
Copy link
Contributor

@Kizr Kizr commented Jan 16, 2025

Context

The CSV's we sent to providers after claims had been requested for sampling were lacklustre, they have been improved for more clarity. Additionally all manual actions now create audit logs.

Note that no specific text has been added to identify individual claims as this is currently under review by the design team.

Changes proposed in this pull request

  • Reworks the stored CSV files for sending to providers and their responses
  • Adds audit logging to all manual flows

Guidance to review

  • Log in as Colin
  • Work through the sampling and clawback flows
  • Check the activity logs

Link to Trello card

Manual clawback/sampling processes should create activity logs

Screenshots

image

@Kizr Kizr requested a review from a team as a code owner January 16, 2025 13:18
@Kizr Kizr self-assigned this Jan 16, 2025
@Kizr Kizr force-pushed the ba/add-activity-logging branch 2 times, most recently from fd58afa to ac7da82 Compare January 16, 2025 13:33
@Kizr
Copy link
Contributor Author

Kizr commented Jan 16, 2025

I've fixed the issue with uploading the sampling CSV from the data team, but I can't get a new screenshot due to GIAS being down.

@Kizr Kizr marked this pull request as draft January 16, 2025 16:31
@Kizr Kizr force-pushed the ba/add-activity-logging branch 3 times, most recently from 5add0ce to 759c23a Compare January 21, 2025 11:30
@Kizr Kizr force-pushed the ba/add-activity-logging branch from 759c23a to 9befd89 Compare January 21, 2025 12:56
@Kizr Kizr marked this pull request as ready for review January 21, 2025 12:57
@Kizr Kizr requested a review from a team as a code owner January 21, 2025 12:57
@Kizr Kizr merged commit e630990 into main Jan 21, 2025
8 checks passed
@Kizr Kizr deleted the ba/add-activity-logging branch January 21, 2025 14:30
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.

2 participants