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

As a dev, I want to refactor the user page call DashboardPage component. #283

Open
11 tasks
Tracked by #186
SamuelPelletierEvraire opened this issue Oct 10, 2024 · 5 comments · May be fixed by #333
Open
11 tasks
Tracked by #186

As a dev, I want to refactor the user page call DashboardPage component. #283

SamuelPelletierEvraire opened this issue Oct 10, 2024 · 5 comments · May be fixed by #333
Assignees
Labels
enhancement New feature or request Refactor For refactoring task

Comments

@SamuelPelletierEvraire
Copy link
Contributor

SamuelPelletierEvraire commented Oct 10, 2024

Description

Refactor the user information that wasn't include previously in the project

Prototype

Dashboard design

image

Header button to go to dashboard

Header button to go to dashboard

Animation on dashboard

Animation dashboard page

Tasks

Implement the dashboard page for user information.

Acceptance Criteria

  • The page contain the inspector name (for now the username).
  • The page contain the email address (Not show but coded for later).
  • The page contain the role of the user (Not show but coded for later).
  • The page contain the location of the inspector (For now this information is not include into our database).
  • The page contain the number of inspection done by the user (For now this information is not include into our database).
  • The page contain inspections done and not finish by the user.
  • The inspection not finished by the inspector appear at the top of the list of inspection and are identified with a badge.
  • A search bar is integrated to search on the user inspection to facilitate the research if the inspection have a lot of inspection.
  • Pagination is include if there are more than 10 inspection related to the user. (Not on the design so make sure to check with the team before designing it).
  • The loading of the inspection are not made all in one request. They are made by pagination to reduce latency.
  • When the user click on an element of the label list, the information page should appear showing the information of the label.

Additional information

Figma design (Please note that the dimension on the design are not accurate)

https://www.figma.com/design/wZAiODRsAfqa8cAxJtQAYr/Untitled?node-id=0-1&t=jbtVtLENKlvQoKjX-1

@SamuelPelletierEvraire SamuelPelletierEvraire added the enhancement New feature or request label Oct 10, 2024
@SamuelPelletierEvraire
Copy link
Contributor Author

@Francois-Werbrouck can it be possible to add this since it is not a sensitive data? (I need more data on user to full fill the dashboard page more professionally

  • The page contain the location of the inspector (For now this information is not include into our database)

@SamuelPelletierEvraire
Copy link
Contributor Author

@Francois-Werbrouck

Can it be possible to include picture only for this page. The picture would be loaded by pagination request of 10 to reduce the the loading time. Since inspector would not have 2000 label on is account I thinks this would not be a big difference. Feel free to rectify me if this is not a good idea.

@SamuelPelletierEvraire
Copy link
Contributor Author

@Francois-Werbrouck @k-allagbe is this possible to include to put more information on the dashboard?

  • The page contain the number of inspection done by the user (For now this information is not include into our database).

@Francois-Werbrouck
Copy link

to put more information on the dashboard?

Yes but is it relevant information? (Number of inspection can be calculated easily)


Can it be possible to include picture only for this page.

Yes one of the 'restriction' we currently have is the picture are store inside the the user's container. We can easily fetch the user's own pictures.

I'm still uncertain about how we want to handle the pagination. Fetching the data from the DB is quicker in one go. However, loading the blobs (picture files) could be done by batch of 10.

I dont know about the best practice of pagination. There probably exists design pattern for such tasks.


Can it be possible to add [the location of the user] since it is not a sensitive data? (I need more data on user to full fill the dashboard page more professionally

I mean, sure; However that means requiring the user to fill in that data when creating its account. We have no way of knowing the user's name, location, email, etc. if it isn't provided to us manually. I get you want the dashboad to look more "professional", but adding those field doesn't bring much value in the eyes of the Inspector nor does it bring value to our prototype. I'd leave displaying extra data for later on in the life cycle of our project.

@k-allagbe
Copy link
Member

k-allagbe commented Oct 10, 2024

I agree with @Francois-Werbrouck. Let's keep in mind the objective of just giving a more professional look to what we currently have, without requiring too many additional features or core data modification.

@k-allagbe k-allagbe mentioned this issue Oct 16, 2024
18 tasks
@Youllou Youllou self-assigned this Nov 14, 2024
@Youllou Youllou moved this to In progress in FertiScan Nov 14, 2024
@Youllou Youllou linked a pull request Nov 19, 2024 that will close this issue
@Youllou Youllou linked a pull request Nov 19, 2024 that will close this issue
@Youllou Youllou added the Refactor For refactoring task label Nov 21, 2024
@gh-app-for-workflows gh-app-for-workflows bot moved this from In progress to In review in FertiScan Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Refactor For refactoring task
Projects
Status: In review
Development

Successfully merging a pull request may close this issue.

4 participants