Skip to content

fixes to the enterprise_over_limit internal alert - #6592

Open
metmarkosaric wants to merge 3 commits into
masterfrom
enterprise_over_limit
Open

fixes to the enterprise_over_limit internal alert#6592
metmarkosaric wants to merge 3 commits into
masterfrom
enterprise_over_limit

Conversation

@metmarkosaric

Copy link
Copy Markdown
Contributor

it's been a while since we touched this alert so I've taken the liberty to get some ai help to bring it up to date with teams, CRM etc. main changes:

  • do not send multiple alerts when the same team goes over the limit (right now it sends one email per team owner)
  • do not send the alert when someone hits the site limit, only when someone goes above the site limit (right now it sends the alerts and starts grace period when someone hits the limit)
  • show what's over the limit and add a link to the team to make it easier to handle the case

@RobertJoonas

Copy link
Copy Markdown
Contributor

@metmarkosaric I've adjusted the code and the email template a bit. Everything that you mentioned in the PR description still works. Here's how the email now looks like:

image

@metmarkosaric

Copy link
Copy Markdown
Contributor Author

That’s great, thank you @RobertJoonas! Let me know if I need to do anything else

@github-actions

Copy link
Copy Markdown
Preview environment👷🏼‍♀️🏗️
PR-6592

@RobertJoonas
RobertJoonas requested a review from a team August 12, 2026 07:46
@RobertJoonas

Copy link
Copy Markdown
Contributor

That’s great, thank you @RobertJoonas! Let me know if I need to do anything else

From our side it's ready then. Would just like to get it reviewed by another developer before merging.

@RobertJoonas
RobertJoonas added this pull request to the merge queue Aug 12, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants