-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Story] Change user avatar within EX #2039
Comments
Should we also allow removing the current avatar? |
This is a a good idea. I would say if we do it for this screen, we should do it for the room avatar as well. @VolkerJunginger what do you think? Should we offer the user the option to remove their avatar now? |
Is this two things? My avatar and the avatar in a room? |
That's already the case on Android, that's why I asked since we're more or less copying the code for the UI and event logic and adapting it to different Rust calls 🤔 . |
@VolkerJunginger I think he meant for removing the avatar of a room in its settings? FWIW I uploaded a quick PR in the Rust SDK to allow removing the avatar, in case we want to do it en EX. |
If this is the same thing effort-wise. Sure. |
Description
Figma link: TBD
Acceptance criteria
Size estimate
S
Dependencies
None
Preconditions
None
Out of scope
?
Open questions
?
Subtasks
Android
iOS
Rust
Client.remove_avatar
function matrix-org/matrix-rust-sdk#2550The text was updated successfully, but these errors were encountered: