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.

Sign up for GitHub

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

Jump to bottom

Sami/member list table #67

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Sami/member list table #67

wants to merge 8 commits into from

Conversation

9samioh
Copy link
Contributor

@9samioh 9samioh commented Mar 11, 2023

What's new in this PR

Description

  • Created the members table on the managers "member" view (using redux!!)
  • added search capabilities, searches by any strings in the display name
  • handles row click of the table, opens Marcos' modal for editing member information
  • the modal uses the old queries (TODO: update to use redux) but can edit member information : name + email
  • also TODO: add last name field on the edit member modal @ Marcos

Screenshots

image
image

How to review

Next steps

Relevant Links

Online sources

Related PRs

CC: @gregoriiaaa

@9samioh 9samioh requested a review from gregoriiaaa March 11, 2023 22:42
@vercel
Copy link

vercel bot commented Mar 25, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
berkeley-student-cooperative ❌ Failed (Inspect) Apr 6, 2023 1:51am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gregoriiaaa gregoriiaaa Awaiting requested review from gregoriiaaa

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@9samioh