[FIX] DMs being created with username instead of user's name

[gabriellsh]

Proposed changes (including videos or screenshots)

Issue(s)

closes #23093

Steps to test or reproduce

Further comments

[ankar84]

@gabriellsh hey!
What do you think is that PR fix that issue #23093 ?
Or it fix different issue?

[gabriellsh]

@gabriellsh hey! What do you think is that PR fix that issue #23093 ? Or it fix different issue?

hey @ankar84 ! Reading through that issue, I believe this is the fix. Thanks for the tip! I'll close it when this is merged.

[sampaiodiego]

do you mind adding tests @gabriellsh ? 🤗

thx for the fix anyways

[sampaiodiego]

found a missing await on the Promise.all call..

added other suggestions for improving readability and consistency :)

[sampaiodiego]

oh, awesome job overall btw.. that's a single line fix that required a lot of tests to prove it works.. thanks for this great improvements

[a89h4ya]

#11598 says that the issue is also in 3.18. Shouldnt the fix be backported to that major version too?