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

ADMIN REDESIGN: Standardized User Management - Admins & Super Admins #1885

Closed
palisadoes opened this issue Apr 6, 2024 · 27 comments
Closed
Assignees
Labels
feature request no-issue-activity No issue activity ui/ux issue related and being worked with the figma file of the Admin UI

Comments

@palisadoes
Copy link
Contributor

palisadoes commented Apr 6, 2024

Is your feature request related to a problem? Please describe.

The User management screens for Admin and Super Admin are very different

  1. Admin

    image

  2. Super Admin

    image

Describe the solution you'd like

  1. Refer to our official figma design for the items below:

    1. https://www.figma.com/file/WksD4uIa5sQJMIk3pR4l0c/Talawa-Design-System-(Official)?type=design&node-id=307-212&mode=design&t=qM7o7OZKjp59AX7Z-0
  2. We need both to resemble the desired design in our official Figma. Where organization specific modifications are specified in the Organizations tab

    image

  3. The Organization People and Community Users screen must match that of the existing Organization People screen:

    1. with the Action column removed

    2. clicking on a row must invoke the User design

      image

  4. When the Organization tab is selected:

    1. The sort, search, and filtering options must be removed
    2. The only options should be for changing:
      1. the blocking status
      2. roles
    3. In the Admin screen, only the current organization must be visible to prevent confusion
    4. In the Super Admin screen, all organizations must be visible

    image

  5. You will also need to standardize the titles.

    1. In the Admin screens we should only be referring to members, not people and users. The word Talawa should be removed
    2. In the Super Admin screens we should only be referring to users.

    image

    image

Describe alternatives you've considered

  • N/A

Approach to be followed (optional)

  • See above

Additional context

Assistance:

  1. Please work with the following for guidance
    1. @aashimawadhwa
    2. @rishav-jha-mech

Related Issues:

  1. ADMIN REDESIGN: redesign the user profile for the talawa admin  #1615

Potential internship candidates

@Doraemon012
Copy link
Member

@palisadoes , I would like to work on this issue. Please assign it to me.

@Doraemon012
Copy link
Member

Doraemon012 commented Apr 8, 2024

@palisadoes @aashimawadhwa @rishav-jha-mech

I had a doubt regarding the instructions provided in the issue. I need a bit of clarification regarding point 4.

In part 4 of the point, it's mentioned that

   4. In the Super Admin screen, all organizations must be visible

In the figma design, the page categorizes organizations as created by the user, joined by the user, etc. When you mention "all organizations," are we supposed to display every single organization regardless of the category? If so,how will we be able to view the different types of organizations (like the organizations joined by user, created by user, etc.)?

Additionally, the part 2 mentions that

   2. The only options should be for changing:
      
      1. the blocking status
      2. roles

Since these options are organization-specific, should we modify the UI to display these options corresponding to every organization on the page?

This is the UI of the page as per the Figma design:

320216734-19d192de-4062-4578-a102-001538b0c707

@palisadoes
Copy link
Contributor Author

palisadoes commented Apr 8, 2024

  1. When a Super Admin is looking at the organization tab, they should see all available organizations
  2. When an Admin is looking at the organization tab, they should only see organizations they are admins for
  3. Yes, you should update the card design for blocking and roles. There should be no manage option as there is now

The image you posted is not visible

@Doraemon012
Copy link
Member

Doraemon012 commented Apr 9, 2024

@palisadoes @aashimawadhwa @rishav-jha-mech

I've prepared A design based on the instructions provided above.

Organization _ Member _ Organization

Please review the design and confirm if it aligns correctly with the instructions.


I have a small suggestion though. According to the Figma designs shared, the user flow for a superadmin has a specific page for viewing all organizations. Now, if we're showing all organizations in the organizations tab within the user section, what would be the purpose of it? Wouldn't this be duplicating information?

As this page falls under the user section, shouldn't it display organizations related to the specific user instead?

Based on the current figma designs, here's the user flow for a superadmin:

Flow

@palisadoes
Copy link
Contributor Author

Thanks for the observations.

  1. The card design should display the user's role for the organization so that it's immediately known. The information on the number of members and admins isn't necessary.
  2. There should be an option to remove the member from the organization's list

Related to the organizations to view:

  1. If we only show the user's orgs, then the admin may not have privileges to alter authorization permissions in the other orgs.
  2. If we show the orgs the admin has management for, then the admin can add users to other orgs.

Based on this, it's best to focus on the current org in Admin and leave the multi-org options to the Super Admin. This would mean that the tab would be labeled Organization in Admin, but Organizations in Super Admin

@Doraemon012
Copy link
Member

@palisadoes @aashimawadhwa @rishav-jha-mech

I have changed the design of the card as per the requirements. Please take a look.

image

I have created this page design for superadmins:

Organization _ Member _ Organization(1)

Whereas for admins, only current organization will be displayed:

Organization _ Member _ Organization(2)

Please review the design and confirm if it aligns correctly with the requirements.

@palisadoes
Copy link
Contributor Author

It may be better to have a standardized dropd own look and feel for this.

  1. Role
    1. User
    2. Admin
    3. SuperAdmin
  2. Status
    1. Active
    2. Blocked
  3. Member
    1. Yes
    2. No

The disadvantages to this approach are:

  1. we don't have the impact of colors. Is that possible in a dropdown?
  2. We lose some of the organization information. I don't think this is necessary. We don't need the statistics for these screens. It's more relevant in the select organization screen for Admins / Super Admins

Can you think of a better layout?

image

@pranshugupta54
Copy link
Member

Since this issue involves working with People page, we can also add a task:

  • Add "Blocked Members" option to Filter on People page (Instead of "Search by Role", can make it "Filter")
image

@palisadoes
Copy link
Contributor Author

@Doraemon012

  1. Do you need additional guidance?
  2. Let's leave the filtering out of scope for now. Though a valid suggestion, it would delay the base implementation

@Doraemon012
Copy link
Member

@palisadoes

I've created a design based on the card layout with dropdowns as suggested. Here's the image:

Screenshot from 2024-04-13 00-17-24

Following the recommendations, I've incorporated three dropdowns:

  • Role
  • Status
  • Member

Additionally, I've used colors to enhance intuitiveness, such as green indicating "yes" and red indicating "no."

As the roles are assigned on an organization basis, I suggest disabling the role dropdown when the user is not a member of the particular organization.

Screenshot from 2024-04-13 00-18-14

While omitting the search and filter functionality, this would be the overall design of the page:

Organization _ Member _ Organization(3)

Please take a look.

@palisadoes
Copy link
Contributor Author

  1. Thanks for the update. This is satisfactory. Please go ahead.
    1. Make sure that long strings for the organization name and address are truncated with elipses (...) with a mouse over just in case.
  2. I expect @aashimawadhwa and @rishav-jha-mech to have comments. Hopefully they will be minor

@pranshugupta54
Copy link
Member

pranshugupta54 commented Apr 12, 2024

As the roles are assigned on an organization basis, I suggest disabling the role dropdown when the user is not a member of the particular organization.

It's member detail for that particular user in that specific Organization so we should only show one Org instead of showing all Orgs even when viewer is Non-Admin for other orgs. So where do we get disabled roles?

@palisadoes
Copy link
Contributor Author

As the roles are assigned on an organization basis, I suggest disabling the role dropdown when the user is not a member of the particular organization.

It's member detail for that particular user in that specific Organization so we should only show one Org instead of showing all Orgs even when viewer is Non-Admin for other orgs. So where do we get disabled roles?

I overlooked that.

@Doraemon012 the logic would mean that we would never see tiles for disabled roles. You'll need to update this in your PR.

Copy link

This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue.

@github-actions github-actions bot added the no-issue-activity No issue activity label Apr 23, 2024
@Cioppolo14
Copy link

Unassigning due to no activity or open PR

@Doraemon012
Copy link
Member

@Cioppolo14 @palisadoes I am working on the issue. I have already created and fianilazied the figma designs as visible in the above comments. I have also raised a related PR in talawa-api repository.
I will complete this as soon as possible. Please reassign this to me.

@palisadoes
Copy link
Contributor Author

Are you still working on this?

@Doraemon012
Copy link
Member

@palisadoes I apologise for the delay. I was having my end semester exams.
The code is almost ready. I'll raise a PR within next two days.

@Doraemon012
Copy link
Member

@palisadoes Please take a look at the screen recording below.

Does this functionality align with the requirements of the issue ?

Screencast.from.05-11-2024.09.30.08.PM.webm

@palisadoes
Copy link
Contributor Author

Getting closer.

  1. As this is an Admin specific screen, you should only see one organization listed. That would be the organization being edited.
  2. You'll need to add translations for the dropdowns.
  3. What does "Conflicting Entry" mean in this context? It's not obvious to the user.

@Doraemon012
Copy link
Member

1. As this is an Admin specific screen, you should only see one organization listed. That would be the organization being edited.

Sorry for not being clear. The screen shown in the screen recording is the one for a super admin. As discussed, the super admin screen will contain all the organizations, while the admin screen will show only the selected organization.

2. You'll need to add translations for the dropdowns.

Sure, working on it.

3. What does "Conflicting Entry" mean in this context? It's not obvious to the user.

This is due to a bug in my code, I will fix it.

Copy link

This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue.

@github-actions github-actions bot added the no-issue-activity No issue activity label May 22, 2024
@palisadoes
Copy link
Contributor Author

@Doraemon012 I closed your PR due to inactivity. We really need the output to match the Figma design. Are you willing to give it another try?

@Doraemon012
Copy link
Member

@palisadoes I apologize for the inactivity. I was waiting for the reviewer's feedback on the Figma design.

I'll fix the conflicting files first, and then we can either discuss the design or wait for the reviewer's response.

Please reopen my PR.

@palisadoes
Copy link
Contributor Author

Thanks. Please reproduce the Figma design aesthetic for now.

@github-actions github-actions bot removed the no-issue-activity No issue activity label Aug 23, 2024
@Doraemon012 Doraemon012 self-assigned this Aug 28, 2024
Copy link

This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue.

@github-actions github-actions bot added the no-issue-activity No issue activity label Sep 18, 2024
@palisadoes
Copy link
Contributor Author

The GSoC projects have addressed this issue

@palisadoes palisadoes closed this as not planned Won't fix, can't repro, duplicate, stale Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request no-issue-activity No issue activity ui/ux issue related and being worked with the figma file of the Admin UI
Projects
Development

No branches or pull requests

4 participants