Skip to content

fix: use Promise.allSettled instead of Promise.all to fetch users in …#489

Merged
kristersd merged 1 commit intomainfrom
kristersd--promise-allsettled-emojimod
Jul 8, 2025
Merged

fix: use Promise.allSettled instead of Promise.all to fetch users in …#489
kristersd merged 1 commit intomainfrom
kristersd--promise-allsettled-emojimod

Conversation

@kristersd
Copy link
Copy Markdown
Contributor

@kristersd kristersd commented Jul 8, 2025 •
edited
Loading

Use Promise.allSettled instead of relying on Promise.all as it can fail for some users. Log the user information in bot logs.

@kristersd kristersd merged commit 26b6251 into main Jul 8, 2025
2 checks passed
@kristersd kristersd deleted the kristersd--promise-allsettled-emojimod branch July 8, 2025 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@kristersd