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

Overhaul Edit Member Display #102

Open
5 tasks
Leundai opened this issue Jul 5, 2021 · 0 comments · May be fixed by #107
Open
5 tasks

Overhaul Edit Member Display #102

Leundai opened this issue Jul 5, 2021 · 0 comments · May be fixed by #107
Assignees

Comments

@Leundai
Copy link
Member

Leundai commented Jul 5, 2021

Description

Currently, we lead members to a different page when they click on a member. We also want a modal to pop up that will have all of the member's data populated. Where those with permissions can edit specific fields just like the current existing page.

image
image

NOTE: This relies on #101 where edit member modal will reside in this given location:
image

Learning objectives

  • Modal implementations
  • Working in parallel with another dev
  • Stretch Goal: Make this new component also compatible with other types of data. Not just member data like chapters and projects

Todo

These points are a rough guideline. Please feel free to discuss with others on the team about the best way to design this component!

  • Research libraries or use already imported libraries that might contain ways to facilitate modal implementation
  • Create files and implement profile component
  • Replace the current profile page with the new profile component
  • Connect to functions in ../utils/apiWrapper.js
  • Once Overhaul Member Grid #101 is implemented create an issue or in your PR connect to the new table and display it as a modal

Useful links

@Leundai Leundai changed the title Rehaul Edit Member Display Overhaul Edit Member Display Jul 5, 2021
@amit-sawhney amit-sawhney linked a pull request Jul 15, 2021 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants