Skip to content
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

Open
01bps opened this issue Feb 21, 2025 · 12 comments
Open

Enhance User Profiles with Display Picture, Name & Designation #81

01bps opened this issue Feb 21, 2025 · 12 comments
Assignees

Comments

@01bps
Copy link
Contributor

01bps commented Feb 21, 2025

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!

@mdxabu
Copy link
Member

mdxabu commented Feb 22, 2025

In my view, the majority of users will be patients and volunteers.

@01bps
Copy link
Contributor Author

01bps commented Feb 22, 2025

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.

@pradeeban
Copy link
Member

pradeeban commented Feb 23, 2025

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.

@ishaanxgupta
Copy link
Contributor

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
which navigates to /userprofile section will be good
@01bps

@01bps
Copy link
Contributor Author

01bps commented Feb 23, 2025

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 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!

@pradeeban
Copy link
Member

Yes, that sounds logical!

@01bps
Copy link
Contributor Author

01bps commented Feb 23, 2025

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 which navigates to /userprofile section will be good @01bps

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.

@ishaanxgupta
Copy link
Contributor

ishaanxgupta commented Feb 23, 2025

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 which navigates to /userprofile section will be good @01bps

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.
You can start with your implementation, would love to refine and make the UI better

@01bps
Copy link
Contributor Author

01bps commented Feb 23, 2025

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 which navigates to /userprofile section will be good @01bps

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. You can start with your implementation, would love to refine and make the UI better

I see what you mean!
My initial thought was to keep it on the same page for simplicity, but I’ll make sure the layout doesn’t feel cluttered.
I’ll start with the implementation, and we can refine it as we go. Looking forward to improving the UI together!

@ishaanxgupta
Copy link
Contributor

Hi @01bps,
Did you start with the implementation? I had a awesome idea on the UI part
I can like setup the base and you can build on the functionality on the UI.
Looking forward to collaboration

@01bps
Copy link
Contributor Author

01bps commented Feb 28, 2025

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!

@ishaanxgupta
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants