[FIX] Updated names in thread messages.

[Darshilp326]

Proposed changes (including videos or screenshots)

Thread messages would now include updated names.

name-update-pr.mp4

Issue(s)

Closes #20633

Steps to test or reproduce

Further comments

[Darshilp326]

Actually, none of the messages contains the updated name in the database. So after this PR, it would fix that issue also.

[jsalatiel]

Would this fix #18270 by any chance?

[KevLehman]

Hey @Darshilp326 ! Thanks again for contributing. Even when the PR solves the issue, I don't think it's the best way of doing it. Currently, we're not storing the name of the user in the database, just the username. This facilitates updates and reduces the amount of data that we need to care about when a user changes his profile information.

I was taking a look at the getThreadMessages file, which controls the data we return to the client. In that place, we're missing a normalization step, which is the one in charge of adding the name of the user to each payload, as you can see here

I can help you with the changes if you want. Thanks again!

[rodrigok]

Missing normalization on other threads methods and rest apis

[Darshilp326]

@rodrigok Onto it!

[Darshilp326]

@rodrigok Hello! Can you tell which API/methods are you exactly referring to?

[Darshilp326]

@KevLehman @rodrigok Sorry sir, I missed this PR. Can you please review it?

[CLAassistant]

All committers have signed the CLA.