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 Transaction ID datadog link to va.gov-specific alerts #21094

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cilestin
Copy link
Contributor

@cilestin cilestin commented Mar 5, 2025

Summary

Workflow from taking a claim ID datadog link, copying a transaction id from the detail, then pasting it back into the search felt not-great. In the cases of va.gov autoestablish claim errors, add a link to the transaction id log search directly to ease some suffering.

Related issue(s)

Testing done

  • New code is covered by unit tests

Before, only the claim ID would have a link to datadog.

Now, both the claim ID and its associated Transaction ID both have links to datadog.

What areas of the site does it impact?

Claims API slack alerting. Nothing consumer-facing.

Acceptance criteria

  • I fixed|updated|added unit tests and integration tests for each feature (if applicable).
  • No error nor warning in the console.
  • Events are being sent to the appropriate logging solution
  • Documentation has been updated (link to documentation)
  • No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs
  • Feature/bug has a monitor built into Datadog (if applicable)
  • If app impacted requires authentication, did you login to a local build and verify all authenticated routes work as expected
  • I added a screenshot of the developed feature

@cilestin cilestin added the claimsApi modules/claims_api label Mar 5, 2025
@va-vfs-bot va-vfs-bot temporarily deployed to add-transaction-id/main/main March 5, 2025 01:00 Inactive
stiehlrod
stiehlrod previously approved these changes Mar 5, 2025
@stiehlrod stiehlrod self-requested a review March 5, 2025 15:02
@cilestin cilestin force-pushed the add-transaction-id branch from b420bc8 to ee0b447 Compare March 6, 2025 00:47
@va-vfs-bot va-vfs-bot temporarily deployed to add-transaction-id/main/main March 6, 2025 00:48 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
claimsApi modules/claims_api test-failure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants