Skip to content

update feedback api alarm threshold#132

Merged
AnJuHyppolite merged 1 commit intodevfrom
feedback-api-alarm-threshold-1395
Apr 2, 2026
Merged

update feedback api alarm threshold#132
AnJuHyppolite merged 1 commit intodevfrom
feedback-api-alarm-threshold-1395

Conversation

@AnJuHyppolite
Copy link
Copy Markdown
Contributor

@AnJuHyppolite AnJuHyppolite commented Apr 2, 2026

Description

This PR increases the Feedback Widget API alarm threshold for the 5xx error to reduce alarm noise.

Ticket

[chore]: Update Feedback Widget Alarm Threshold

@AnJuHyppolite AnJuHyppolite force-pushed the feedback-api-alarm-threshold-1395 branch from 676f7e9 to 6d0cc74 Compare April 2, 2026 19:05
template.hasResourceProperties('AWS::CloudWatch::Alarm', {
MetricName: metricName,
MetricName: '4XXError',
Threshold: 2,
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this threshold need to be updated as well?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 4XX alarm hasn’t been triggering, so I was thinking of leaving it as-is. I can update it too if you think it best.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thats ok, just wondering

@AnJuHyppolite AnJuHyppolite merged commit 62cbfa9 into dev Apr 2, 2026
1 check passed
@AnJuHyppolite AnJuHyppolite deleted the feedback-api-alarm-threshold-1395 branch April 2, 2026 20:00
AnJuHyppolite added a commit that referenced this pull request Apr 3, 2026
<!-- Please complete the following sections as necessary. -->

### Description

<!-- Summary of the changes, related issue, relevant motivation, and
context -->

Merges the following PRs to main:
- #132
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