-
Notifications
You must be signed in to change notification settings - Fork 3
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
Replaces boring avatar with avvvatars-react package. #2186
Conversation
…elated to the old implementation.
@praveen-murali-ind _a please review. Let's do a beta release and integrate it into one of the products. |
@deepakjosp CI is red. |
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.
@deepakjosp UI looks good to me. @josephmathew900 _a Could you please review.
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.
@deepakjosp _a LGTM
Published a beta release from this branch. |
…logic to render fist letters of firstname and lastname.
@deepakjosp CI is red. |
@josephmathew900 _a I've made changes as suggested by Neeraj in the ticket. Please review. |
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.
@deepakjosp _a Added a minor suggestion. Please check. Rest LGTM.
Fixes https://github.com/bigbinary/neeto-engineering-ui-ux/issues/275
Description
Replaces the boaring-avatar component with avvvatars-react component as the fallback avatar.
Checklist
I have made corresponding changes to the documentation.I have updated the types definition of modified exports.I have added tests that prove my fix is effective or that my feature works.I have added properdata-cy
anddata-testid
attributes.patch
/minor
/major
- If package publishis required).
Reviewers