Skip to content
This repository was archived by the owner on Sep 20, 2024. It is now read-only.

Conversation

Shyrro
Copy link
Contributor

@Shyrro Shyrro commented Sep 28, 2022

Signed-off-by: Shyrro [email protected]

Pull request checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes
    /start features)

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Does this introduce a breaking change?

  • Yes
  • No

Other information

This PR introduces the CAvatar component alongs with the CAvatarBadge and CAvatarGroup.

@vercel
Copy link

vercel bot commented Sep 28, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
chakra-ui-vue-next ✅ Ready (Inspect) Visit Preview Sep 29, 2022 at 4:32PM (UTC)
chakra-ui-vue-next-playground ✅ Ready (Inspect) Visit Preview Sep 29, 2022 at 4:32PM (UTC)

Copy link
Collaborator

@codebender828 codebender828 left a comment

Choose a reason for hiding this comment

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

This looks great @Shyrro . Thank you. Just need to add some changes for the component label props for debugging.

@Shyrro
Copy link
Contributor Author

Shyrro commented Sep 29, 2022

This looks great @Shyrro . Thank you. Just need to add some changes for the component label props for debugging.

Thanks for the review ! I'll add these asap !

@Shyrro Shyrro force-pushed the feature/avatarComponent branch from 79fbf1e to 6a02716 Compare September 29, 2022 16:25
@Shyrro
Copy link
Contributor Author

Shyrro commented Sep 29, 2022

Hi @codebender828

Just updated the PR with the labels and the updated snapshots accordingly !

@codebender828
Copy link
Collaborator

Thank you @Shyrro

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants