Skip to content

fix: catch error when no users are found#17308

Merged
keithwillcode merged 1 commit into
mainfrom
fix/eventType-error
Oct 24, 2024
Merged

fix: catch error when no users are found#17308
keithwillcode merged 1 commit into
mainfrom
fix/eventType-error

Conversation

@sean-brydon
Copy link
Copy Markdown
Member

What does this PR do?

Fixes: https://calcom.sentry.io/issues/6017728159/

@dosubot dosubot Bot added teams area: teams, round robin, collective, managed event-types 🐛 bug Something isn't working labels Oct 24, 2024
@keithwillcode keithwillcode added consumer core area: core, team members only labels Oct 24, 2024
gettingRoundRobinHostsToReassignLogger
);
} catch (error) {
if (error instanceof Error && error.message === ErrorCode.NoAvailableUsersFound) {
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This isnt actually an error we care about here so we just set availableUsers to empty

@graphite-app graphite-app Bot requested a review from a team October 24, 2024 20:30
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Oct 24, 2024

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (10/24/24)

1 reviewer was added to this PR based on Keith Williams's automation.

@github-actions
Copy link
Copy Markdown
Contributor

E2E results are ready!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug Something isn't working consumer core area: core, team members only ready-for-e2e teams area: teams, round robin, collective, managed event-types

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants