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

fix(user): fix changeProfilePicture #1418

Closed
wants to merge 1 commit into from
Closed

fix(user): fix changeProfilePicture #1418

wants to merge 1 commit into from

Conversation

aanev
Copy link
Contributor

@aanev aanev commented Aug 6, 2016

  • use async to simplify callbacks
  • use fs.unlink to delete old picture once the user profile is updated

Fixes #1415

@aanev
fix(user): fix changeProfilePicture
e05be44 
* use async to simplify callbacks

* use fs.unlink to delete old picture once the profile is updated

Fixes #1415
@lirantal
Copy link
Member

lirantal commented Aug 6, 2016

Thanks @aanev
Can you replace async with a promise-based approach?

@lirantal lirantal self-assigned this Aug 6, 2016
@lirantal lirantal added this to the 0.5.0 milestone Aug 6, 2016
@aanev aanev closed this Aug 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@lirantal lirantal

Labels
issue:bug platform:node
Projects
None yet
Milestone
0.5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@aanev @lirantal