-
Notifications
You must be signed in to change notification settings - Fork 12
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
Enhance User Profiles with Display Picture, Name & Designation #81
Comments
In my view, the majority of users will be patients and volunteers. |
Thanks for pointing it out! The user base is indeed two fold—on one side, we have marginalised individuals, and on the other, we have volunteers and researchers, particularly from the University of Alaska System. |
It is a cosmetic feature (i.e., something that does not have a solid use). But I cannot be too sure. I am not a psychologist. Such personalization may help the marginalized folks connect with the application better. Go for it, @01bps. As @mdxabu noted, you can skip the Designation. I think it is irrelevant and maybe harmful, given many of these individuals maybe unemployed. Name and picture are great though. A generic "User" vs "Social Worker" vs "Admin" is ok. But that will need a careful mapping. I think, the separation of patient/user vs admin is clear. Currently social workers are not included as a separate category. So, anyone who is not an Admin is going to default to "User." I don't think it adds much quality to the user experience. Best to skip. Sorry, had to think that out loud. |
I get that this feature like helps in connecting users with the platform but if we have a display picture we should also have a option to edit it also not keeping this user image on home page and having a /userprofile route in a sidebar or a avatar option on the top right |
I understand the concern regarding designations, and I’ll skip that part as suggested. Keeping it to name and picture makes sense for personalization while avoiding potential issues. Regarding roles, I see the reasoning behind defaulting non-admins to "User." Given that social workers are not currently a separate category, I’ll keep the existing structure intact. I’ll proceed with the updates accordingly. Thanks again! |
Yes, that sounds logical! |
Thanks for the suggestion! The option to change the display picture is already in the plan, so users will be able to update it when needed. Regarding the /userprofile route, having it would create a new page, whereas I’m thinking of keeping the profile section on the same page to avoid unnecessary redirection. This would make navigation smoother while keeping everything accessible in one place. |
Will it not look odd like we are uploading images and the user's photo is on the top right, ig will look like a application form idk, i think it will be good if there is a diff section only in this page or a diff profile page is good further we can add the feature of change pwd or change username also in profile page. |
I see what you mean! |
Hi @01bps, |
Yes, I’ve started working on it! Will open a PR soon for review with the initial implementation. I’m sure you’ll enhance it further from there. Looking forward to your input! |
Sure sounds great, waiting for you PR |
Currently, user profiles only display the username, which makes them look quite basic. To enhance the user experience, I propose adding an option for users to upload a display picture, which will make profiles more personalised.
Along with this, instead of just showing the username, we can display the full name and designation(researcher,scientist or professor) at the top centre of the profile page, making it look more professional and informative.
This enhancement can improve the overall look and feel of the platform.
@pradeeban and @mdxabu , would this be a good addition to the project? If so, I’d love to work on it!
The text was updated successfully, but these errors were encountered: