bcgov / lcfs

An online application for fuel suppliers to manage their compliance obligations under the Low Carbon Fuels Act
Apache License 2.0
5 stars 3 forks source link

LCFS - View User page #176

Closed justin-lepitzki closed 6 months ago

justin-lepitzki commented 10 months ago

Description: Develop a View User page accessible to both BCeID and IDIR users. This page will be visible upon clicking on a user row in the respective User tables and will display detailed information about the user. The page should also include breadcrumb navigation for enhanced user experience.

Screenshot of View user page in TFRS: Private Zenhub Image

Wireframe of View User Page for LCFS:

Purpose and benefit to user: The View User page serves as a landing page for detailed information about individual users. This will pave the way for future enhancements such as adding a User Activity table to provide a transparent record of user actions within the application.

Acceptance Criteria:

Development Checklist:

Notes:

justin-lepitzki commented 10 months ago

@Kailee-Douglas @airinggov Wireframe for view user page redesign

Kailee-Douglas commented 10 months ago

This should be done with the MUI components? I added the MUI library to UXPin and can see how it deviates from the BC Gov Design System on the DevHub

airinggov commented 10 months ago

Do we need an intermediary page between the user list on the org page and the edit user page? Can we not go from the list directly to the edit user page? For users without admin access they would only see the info on the user list / org page.

justin-lepitzki commented 10 months ago

@airinggov possibly? We would have to figure out where to put other information that would be lost if you didn't have a "view user" page, such as their job title, mobile phone, and user activity. The first two could potentially be added to the user list on the org page (two additional columns) but the user activity column is a larger component that I'm not sure where we could relocate. Thoughts?

justin-lepitzki commented 10 months ago

@Kailee-Douglas @airinggov I think we've settled on keeping this page so can we create a wireframe for displaying the user information. Do we to change the style a bit here? Doesn't have to be drastic but we do have an opportunity to tweak the layout if desired.

Kailee-Douglas commented 10 months ago

@justin-lepitzki your thoughts on this mockup Current mockup for View User page (Issue #176)

justin-lepitzki commented 10 months ago

Looks good @Kailee-Douglas ! Can we make one tweak: change the label 'Role:' to 'Role(s):'

I noticed that when a user has more than one role it still says the singular Role:

Kailee-Douglas commented 10 months ago

Here you are, @justin-lepitzki

More current mockup for View User page (Issue #176)

I can add the UX Pin link after Al looks at it, too. Functionality to highlight for the dev work: columns autoadjusted to fit width of content (within reason), and rows autosorted by newest first. I did not add a filter option, which is present in other tables, and may or may not be valuable here. Also, does it need a download function?

We discussed breadcrumbs earlier. Depending on our decision around that, we can quickly add a breadcrumb on the top.

prv-proton commented 10 months ago

Hey team! Please add your planning poker estimate with Zenhub @AlexZorkin @jig-patel @kevin-hashimoto @prv-proton

AlexZorkin commented 10 months ago

Hey team! Please add your planning poker estimate with Zenhub @hamed-valiollahi @jig-patel @kevin-hashimoto @prv-proton

Grulin commented 7 months ago

Unable to fully test until given access to BCeID user side. IDIR side is visible with no issues.

Grulin commented 7 months ago

IDIR side is visible with no issues.

BCeID side does not load user table on Organization page. Shows error "Request failed with status code 403. Please contact your administrator". Please see screenshot error below:

Image