-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#158457052 User should be able to upload profile picture on signup #22
Conversation
Pull Request Test Coverage Report for Build 420
💛 - Coveralls |
b9d0074
to
cd0a05e
Compare
cd0a05e
to
08d0f71
Compare
08d0f71
to
b30d35c
Compare
public/js/controllers/auth.js
Outdated
localStorage.setItem('username', response.name); | ||
$location.path('/'); | ||
}, (error) => { | ||
console.log(error); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected console statement no-console
public/js/controllers/auth.js
Outdated
localStorage.setItem('username', response.name); | ||
$location.path('/'); | ||
}, (error) => { | ||
console.log(error); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected console statement no-console
$scope.viewImage = function () { | ||
const file = event.target.files[0]; | ||
if (file) { | ||
const fileReader = new FileReader(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'FileReader' is not defined no-undef
}; | ||
$scope.imagePreview = ''; | ||
$scope.viewImage = function () { | ||
const file = event.target.files[0]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected use of 'event' no-restricted-globals
'event' is not defined no-undef
- Redesign avatar modal - Upload image to cloudinary - Test the upload function - edit build and compile gulp tasks [Finishes #158457052]
b30d35c
to
8854942
Compare
}; | ||
$scope.imagePreview = ''; | ||
$scope.viewImage = function () { | ||
const file = event.target.files[0]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected use of 'event' no-restricted-globals
'event' is not defined no-undef
What does this PR do?
Description of Task to be completed?
As a new user, I should be able to upload my desired profile image on signup
How should this be manually tested?
Clone the repo. Run ’npm install’ and run ’npm start’. Visit ‘http://localhost:3000/#!/signup’ and click on ‘upload profile picture’
What are the relevant pivotal tracker stories?
#158457052
Any background context you want to add?
Screenshots