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

[AB2D-6124] Add Opt Out Metrics to Slack Message #105

Merged
merged 25 commits into from
Jun 19, 2024

Conversation

Rwolfe-Nava
Copy link
Contributor

@Rwolfe-Nava Rwolfe-Nava commented Jun 10, 2024

🎫 Ticket

https://jira.cms.gov/browse/AB2D-6124

🛠 Changes

Add more information to the lambda completion slack message with metrics about processed opt out information.

ℹ️ Context for reviewers

I want to make sure my approach here is good. I wanted to avoid going to the database multiple times for performance reasons.

✅ Acceptance Validation

Tested on IMPL environment:
Screenshot 2024-06-11 at 1 51 21 PM
Screenshot 2024-06-11 at 1 54 38 PM

🔒 Security Implications

  • This PR adds a new software dependency or dependencies.
  • This PR modifies or invalidates one or more of our security controls.
  • This PR stores or transmits data that was not stored or transmitted before.
  • This PR requires additional review of its security implications for other reasons.

If any security implications apply, add Jason Ashbaugh (GitHub username: StewGoin) as a reviewer and do not merge this PR without his approval.

@Rwolfe-Nava Rwolfe-Nava changed the title [WIP][AB2D-6124] Add Opt Out Metrics to Slack Message [WIP] [AB2D-6124] Add Opt Out Metrics to Slack Message Jun 10, 2024
@Rwolfe-Nava Rwolfe-Nava changed the title [WIP] [AB2D-6124] Add Opt Out Metrics to Slack Message [AB2D-6124] Add Opt Out Metrics to Slack Message Jun 11, 2024
@Rwolfe-Nava Rwolfe-Nava merged commit b32f5e5 into main Jun 19, 2024
5 checks passed
@Rwolfe-Nava Rwolfe-Nava deleted the ab2d-6124-add-opt-out-metrics branch June 19, 2024 00:11
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