Skip to content

Commit

Permalink
fix: getTransactionsByAlert only for transaction_monitoring's alerts
Browse files Browse the repository at this point in the history
  • Loading branch information
liorzblrn committed Dec 19, 2024
1 parent 0c46a45 commit 950de1a
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -349,7 +349,10 @@ export class TransactionControllerExternal {
throw new errors.NotFoundException(`Alert with id ${filters.alertId} not found`);
}

if (!alert.alertDefinition) {
if (
!alert.alertDefinition ||
alert.alertDefinition.monitoringType !== MonitoringType.transaction_monitoring
) {
throw new errors.NotFoundException(`Alert definition not found for alert ${alert.id}`);
}

Expand Down

0 comments on commit 950de1a

Please sign in to comment.