Skip to content
This repository was archived by the owner on Jan 16, 2022. It is now read-only.

fix: incorrect Tooltip import in avatar component#160

Merged
priscilawebdev merged 4 commits intoverdaccio:masterfrom
bighuggies:fix-tooltip-import
Oct 6, 2019
Merged

fix: incorrect Tooltip import in avatar component#160
priscilawebdev merged 4 commits intoverdaccio:masterfrom
bighuggies:fix-tooltip-import

Conversation

@bighuggies
Copy link
Copy Markdown
Contributor

@bighuggies bighuggies commented Oct 5, 2019

Type: fix

The following has been addressed in the PR:

Fixes a rendering issue with the developer avatars in the sidebar as pointed out by @juanpicado:
image

Now:
image

I haven't been able to get snapshots to generate correctly on my machine so the tests are failing atm.

@juanpicado
Copy link
Copy Markdown
Member

I will check it

priscilawebdev
priscilawebdev previously approved these changes Oct 6, 2019
@priscilawebdev
Copy link
Copy Markdown
Contributor

@bighuggies I believe you forgot to update snapshots

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 6, 2019

Codecov Report

Merging #160 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #160   +/-   ##
=======================================
  Coverage   82.97%   82.97%           
=======================================
  Files          97       97           
  Lines         928      928           
  Branches      163      163           
=======================================
  Hits          770      770           
  Misses        141      141           
  Partials       17       17
Impacted Files Coverage Δ
src/components/AvatarTooltip/AvatarTooltip.tsx 83.33% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d2f1f1c...3744455. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants