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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement Application Overview Card and Table #116

Closed
8 tasks done
dandheedge opened this issue Feb 13, 2024 · 2 comments 路 Fixed by #125
Closed
8 tasks done

Implement Application Overview Card and Table #116

dandheedge opened this issue Feb 13, 2024 · 2 comments 路 Fixed by #125
Assignees
Labels
Type: Feature Add a new feature to the system

Comments

@dandheedge
Copy link
Contributor

dandheedge commented Feb 13, 2024

馃搫 Context

Applications page is not displaying the total number of applications nor does it provide a table listing the applications owned by the user.

鉁旓笍 Solution

  1. Create a summary card in the /home page that shows total applications owned by user.
  2. Create a table component in the /applications that shows all applications owned by user.

Inspiration

Screenshot 2024-02-13 at 13 54 29

馃搱 Subtasks

  • Create a card that shows total user's applications on the homepage
  • Create a table that shows only applications owned by user
  • Add test cases for all added logic and UI
  • Add stories for the UI under the workshop project

馃幆 Definition of Done

  • A card component titled My Component accurately displays the total number of applications the user has.
  • Table component that only display the user's Applications
  • Correctly use the query on each component
  • Existing and new unit tests pass successfully
@dandheedge dandheedge added Type: Feature Add a new feature to the system Status: Needs triage Explorer team assessment needed labels Feb 13, 2024
@dandheedge dandheedge self-assigned this Feb 13, 2024
@dandheedge dandheedge changed the title Implement Application Overview Card and Wallet Implement Application Overview Card and Table Feb 20, 2024
@brunomenezes brunomenezes removed the Status: Needs triage Explorer team assessment needed label Feb 26, 2024
@nevendyulgerov
Copy link
Contributor

nevendyulgerov commented Mar 6, 2024

Hey @dandheedge , @brunomenezes , maybe we should display a simplified card with user-owned applications on the home page as well? Right now, we have there two cards for latest inputs and latest applications.

I noticed in the PR for this that the above suggestion is already implemented.

@brunomenezes
Copy link
Collaborator

Hey @dandheedge , @brunomenezes , maybe we should display a simplified card with user-owned applications on the home page as well? Right now, we have there two cards for latest inputs and latest applications.

I noticed in the PR for this that the above suggestion is already implemented.

Exactly, that was the feature discussed: the card that has the info to be displayed on the Home page and the table for the applications page (I think the issue context section needs some updating). @dandheedge, maybe refining the issue a bit more, including where these "new components" will be added, would make things clearer for a person without context or the reviewer of this PR once it is ready.

Also, in the DoD;
I wording update is necessary for the first checkbox. "A card component titled My Component accurately displays the total number of applications the user has."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature Add a new feature to the system
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants