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
[CL-7] Avatar #3153
[CL-7] Avatar #3153
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets take this opportunity to clean up the component and remove most of the legacy logic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me! @Hinton @danielleflinn will have the final approvals 👍
(@rr-bw this will need another merge from master, however)
Looks good to me too. Thanks all |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me except for that outdated comment. Thanks for iterating on this @rr-bw.
f2e5898
Type of change
Objective
Code changes
Screenshots
Before you submit