Skip to content

Implement MessageLikeManager#110

Open
not-so-smart wants to merge 4 commits intogroupme-js:mainfrom
not-so-smart:main
Open

Implement MessageLikeManager#110
not-so-smart wants to merge 4 commits intogroupme-js:mainfrom
not-so-smart:main

Conversation

@not-so-smart
Copy link
Copy Markdown
Member

Message#likes was previously a (string | User)[] 🤮

instead of using an array of `User | string`
we don't get user avatars from member data, so we need a way to show that we don't have this data yet (as opposed to knowing it's empty with `null`)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant