Skip to content

Conversation

@vitormattos
Copy link
Contributor

@vitormattos vitormattos commented Aug 31, 2022

Rebased from: #4737

The RoomAvatar does not provide (yet) a default avatar for a room if it
has not been set.

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
Now the avatar of the other user will be returned when getting the
avatar of a one-to-one conversation.

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
Although the avatar itself is stored in the app data the avatar id and
its version are stored in the database for convenience.

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
The image itself is always got directly from the user avatar, so it is
always up to date. However, as the version is specific to the room
avatar and stored in the database it needs to be explicitly bumped.

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
Signed-off-by: Daniel Calviño Sánchez <[email protected]>
The generic avatar system will not be included in Nextcloud 21, so for
the time being Talk needs to provide its own endpoints for room avatars.

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
@vitormattos vitormattos changed the title Add backend for conversation avatars #4737 Add backend for conversation avatars Aug 31, 2022
Signed-off-by: Vitor Mattos <[email protected]>
@nickvergessen nickvergessen added this to the 💔 Backlog milestone Sep 13, 2022
@nickvergessen nickvergessen added 1. to develop feature: api 🛠️ OCS API for conversations, chats and participants labels Sep 13, 2022
@nickvergessen
Copy link
Member

Replaced by #8333

@nickvergessen nickvergessen deleted the add-backend-for-conversation-avatars-v2 branch December 21, 2022 19:59
@Antreesy Antreesy removed this from the 💔 Backlog milestone Apr 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1. to develop feature: api 🛠️ OCS API for conversations, chats and participants

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants