-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Suggestion: Edit account page #770
Comments
@skyserpent - This is actually very easy to set up. Something we have done internally for our app which is slated for a public release soon. The request makes sense (view user profile, and edit user profile). Along with that, I would actually like to extend the user model with a profile picture as a part of it, and in the process we extend the core user package itself. @fyockm @liorkesos - I could send in a PR for this in the next few days Would it be something you'd be keen to merge? User profile picture, would mean a dependency on the mean upload in the core itself. |
+1
|
Although I think we can have a default profile pic and not require upload
|
pratik - I'd be glad to merge this if you send a PR |
@lior - I will send it by Wednesday evening for sure. On Mon, Sep 15, 2014 at 7:19 PM, Lior Kesos notifications@github.com
|
@liorkesos I can provide with the PR for Edit user profile If you'd like. |
@scrock1987 / @brownman - Feel free to complete/extend/overwrite this in altogetherness.... |
Sauarbh, On Wed, Jan 28, 2015 at 1:09 PM, Saurabh Chawla notifications@github.com
Lior Kesos - http://www.linnovate.net |
PR needed. @pratik60 @scrock1987 |
I think this is a good feature. If anyone wants to work on this, I would be happy to merge. |
Before we had forked the user package, we created a userprofile package to store this info in our project. In hindsight, this is much better stored on the user model itself so we dont have to call |
There should be an edit account page for the boilerplate to make it a better boilerplate. Sort of like how Hackathon-Starter has it.
The text was updated successfully, but these errors were encountered: