keep icon indicating copy to clipboard operation
keep copied to clipboard

fix: [Alert enrichment] when changing status of incident, other enrichments of alerts are gone #5259

Open bbcorp opened this issue 4 months ago • 3 comments

Closes #5259

📑 Description

Alert enrichment: I've noticed when you acknowledge an incident, it acknowledges the alert too by adding the enrichment "status": "acknowledged" but all other previous enrichments are gone. Same goes when you resolve and incident.

Now in batch_enrich function, we merge enrichments before committing them to the db.

✅ Checks

  • [x] My pull request adheres to the code style of this project
  • [ ] All the tests have passed

bbcorp avatar Aug 18 '25 22:08 bbcorp

@bbcorp is attempting to deploy a commit to the KeepHQ Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Aug 18 '25 22:08 vercel[bot]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
keep Ignored Ignored Preview Aug 19, 2025 6:49am

vercel[bot] avatar Aug 19 '25 06:08 vercel[bot]

Thats what I thought too, do you see a way to make it in bulk SQL query anyway? I was thinking we could merge at code level but then I wonder how we could do the SQL in bulk?

bbcorp avatar Aug 19 '25 13:08 bbcorp