Skip to content
This repository has been archived by the owner on Jul 7, 2021. It is now read-only.

fix(profiles): do not set avatar during initialization #700

Merged
merged 1 commit into from
Sep 8, 2020

Conversation

dated
Copy link
Contributor

@dated dated commented Sep 8, 2020

Summary

The default avatar should not be stored.

Checklist

  • Documentation (if necessary)
  • Tests (if necessary)
  • Ready to be merged

@dated dated changed the title fix(profiles): do set avatar during initialization fix(profiles): do not set avatar during initialization Sep 8, 2020
@codecov
Copy link

codecov bot commented Sep 8, 2020

Codecov Report

Merging #700 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #700      +/-   ##
==========================================
- Coverage   64.93%   64.91%   -0.02%     
==========================================
  Files          99       99              
  Lines        2170     2169       -1     
  Branches      240      240              
==========================================
- Hits         1409     1408       -1     
  Misses        678      678              
  Partials       83       83              
Flag Coverage Δ
#unit 64.91% <ø> (-0.02%) ⬇️

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

Impacted Files Coverage Δ
...ages/platform-sdk-profiles/src/profiles/profile.ts 98.78% <ø> (-0.02%) ⬇️

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 03a8ff5...906b0f1. Read the comment docs.

@faustbrian faustbrian merged commit 00b7e51 into master Sep 8, 2020
@faustbrian faustbrian deleted the fix/avatar branch September 8, 2020 14:45
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.

None yet

2 participants