Skip to content
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

Merged
merged 12 commits into from
May 15, 2024

Conversation

deepakjosp
Copy link
Contributor

@deepakjosp deepakjosp commented May 8, 2024

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 verified the functionality in some of the neeto web-apps.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added proper data-cy and data-testid attributes.
  • I have added the necessary label (patch/minor/major - If package publish
    is required).

Reviewers

@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2186 May 8, 2024 15:00 Inactive
@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2186 May 8, 2024 15:03 Inactive
@deepakjosp
Copy link
Contributor Author

@praveen-murali-ind _a please review. Let's do a beta release and integrate it into one of the products.

Copy link
Contributor

neetogit-bot bot commented May 8, 2024

@deepakjosp CI is red.

@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2186 May 8, 2024 15:18 Inactive
@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2186 May 9, 2024 15:18 Inactive
@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2186 May 9, 2024 16:17 Inactive
Copy link
Contributor

@praveen-murali-ind praveen-murali-ind left a 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.

Copy link
Contributor

@josephmathew900 josephmathew900 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@deepakjosp _a LGTM

@neetogit-bot neetogit-bot bot assigned deepakjosp and unassigned josephmathew900 May 10, 2024
@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2186 May 10, 2024 03:39 Inactive
@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2186 May 10, 2024 03:44 Inactive
@deepakjosp
Copy link
Contributor Author

Published a beta release from this branch.
https://github.com/bigbinary/neeto-ui/releases/tag/v6.5.18-beta

…logic to render fist letters of firstname and lastname.
@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2186 May 14, 2024 04:13 Inactive
Copy link
Contributor

neetogit-bot bot commented May 14, 2024

@deepakjosp CI is red.

@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2186 May 14, 2024 04:53 Inactive
@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2186 May 14, 2024 07:24 Inactive
@deepakjosp
Copy link
Contributor Author

@josephmathew900 _a I've made changes as suggested by Neeraj in the ticket. Please review.

cc: @praveen-murali-ind

@neetogit-bot neetogit-bot bot assigned josephmathew900 and unassigned deepakjosp May 14, 2024
Copy link
Contributor

@josephmathew900 josephmathew900 left a 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.

src/components/Avatar.jsx Outdated Show resolved Hide resolved
@neetogit-bot neetogit-bot bot assigned deepakjosp and unassigned josephmathew900 May 14, 2024
@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2186 May 15, 2024 02:48 Inactive
@deepakjosp deepakjosp merged commit 25339c0 into main May 15, 2024
1 check passed
@deepakjosp deepakjosp deleted the 275-boring-avatar-is-not-cutting-it branch May 15, 2024 05:12
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.

4 participants