Skip to content

166 create donors page#175

Open
mehanana wants to merge 3 commits intomainfrom
166-create-donors-page
Open

166 create donors page#175
mehanana wants to merge 3 commits intomainfrom
166-create-donors-page

Conversation

@mehanana
Copy link
Contributor

@mehanana mehanana commented Mar 26, 2026

ℹ️ Issue

Closes #166

📝 Description

Created the donors page to display all donors in a table, and also had a search bar and had buttons for sorting, filtering, and adding new donor.

Briefly list the changes made to the code:

  1. Created the donors page using previously made components (eg. Dropdown) and Chakra UI components (eg. table)
  2. Created tests to ensure everything rendered and buttons rendered the right components when clicked
  3. Compared to the figma designs

✔️ Verification

Here are some screenshots of the donors page. I also ran all my unit tests and they passed.

  1. The donors page itself
  2. When there's multiple pages of donors
  3. Filter by button clicked
  4. Sort by button clicked
  5. Add new donor button clicked --> modal
  6. Error messages for add new donor modal
  7. tests
Screenshot 2026-03-25 173321 Screenshot 2026-03-25 173418 Screenshot 2026-03-25 173439 Screenshot 2026-03-25 173449 Screenshot 2026-03-25 173504 Screenshot 2026-03-25 173512 Screenshot 2026-03-25 220750

🏕️ (Optional) Future Work / Notes

Did you notice anything ugly during the course of this ticket? Any bugs, design challenges, or unexpected behavior? Write it down so we can clean it up in a future ticket!

Didn't notice anything much, just need to add functionality to search, filter by/sort by buttons, and adding new donor

@mehanana mehanana requested a review from nourshoreibah as a code owner March 26, 2026 02:16
@github-actions github-actions bot requested a review from bhuvanh66 March 26, 2026 02:16
github-actions bot added a commit that referenced this pull request Mar 26, 2026
Copy link
Collaborator

@nourshoreibah nourshoreibah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

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

Successfully merging this pull request may close these issues.

Create the donors page

2 participants