Skip to content

Chore: test for change avatar - #26543

Merged
ggazzo merged 7 commits into
developfrom
test/set-new-avatar
Aug 12, 2022
Merged

ggazzo merged 7 commits into
developfrom
test/set-new-avatar

Conversation

@weslley543

Copy link
Copy Markdown
Contributor
  • Add test to change avatar

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@codecov

codecov Bot commented Aug 11, 2022 •

Copy link
Copy Markdown

Codecov Report

Merging #26543 (7722fd6) into develop (50af9b2) will increase coverage by 0.15%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #26543      +/-   ##
===========================================
+ Coverage    38.54%   38.70%   +0.15%     
===========================================
  Files          759      759              
  Lines        18829    18829              
  Branches      1459     1459              
===========================================
+ Hits          7258     7288      +30     
+ Misses       11350    11320      -30     
  Partials       221      221              
Flag Coverage Δ
e2e 38.70% <ø> (+0.15%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Comment thread apps/meteor/client/components/avatar/UserAvatarEditor/UserAvatarEditor.js Outdated
Comment thread apps/meteor/client/components/avatar/UserAvatarEditor/UserAvatarEditor.js Outdated
weslley543 and others added 3 commits August 11, 2022 15:36
…arEditor.js

Co-authored-by: souzaramon <ramon.souza@objective.com.br>
…arEditor.js

Co-authored-by: souzaramon <ramon.souza@objective.com.br>
souzaramon
souzaramon previously approved these changes Aug 11, 2022
@weslley543
weslley543 marked this pull request as ready for review August 11, 2022 20:12
@weslley543
weslley543 requested review from a team as code owners August 11, 2022 20:12
@weslley543 weslley543 added the stat: ready to merge PR tested and approved waiting for merge label Aug 12, 2022
@ggazzo ggazzo removed the stat: ready to merge PR tested and approved waiting for merge label Aug 12, 2022
@weslley543 weslley543 added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Aug 12, 2022
@ggazzo ggazzo added stat: needs QA stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge stat: needs QA labels Aug 12, 2022
@ggazzo
ggazzo merged commit 69475aa into develop Aug 12, 2022
@ggazzo
ggazzo deleted the test/set-new-avatar branch August 12, 2022 18:01
csuadev pushed a commit that referenced this pull request Aug 26, 2022
@murtaza98 murtaza98 mentioned this pull request Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants