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

Add total value donated component #64

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

Add total value donated component #64

wants to merge 16 commits into from

Conversation

aayushh-patell
Copy link
Collaborator

create “Total Value Donated” component

Implementation description

  • A new TotalValueDonated component has been added to the layout page temporarily
  • The fetchTotalDonations function currently displays a static value in the component, but that can be refactored once we figure out how to be able to determine which user is logged in

Steps to test

  1. Visit http://localhost:3000/layout to verify that the manually inputted data is displayed in the component

What should reviewers focus on?

  • Potential Yarn issues caused by removing and regenerating the yarn.lock file

Checklist

  • [Y] My PR name is descriptive and in imperative tense
  • [Y] My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • [Y] I have run the appropriate linter(s)
  • [Y] I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

pahu2353 and others added 16 commits June 10, 2024 20:23
@pahu2353 @Ee-Hsin @leozhang14
basic table and filter created w/ chakra
105aea0 
Co-authored-by: Jordan Kok Ee Hsin <[email protected]>
@leozhang14
expose local web server using ngrok (#43) (#45)
8d819dc 
* Integrated Ngrok

Adjusted docker-compose.yml to run ngrok using my personal authentication token

* add environment variable for ngrok authtoken

* Add ngrok set-up instructions in README.md

* updating ngrok authtoken as variable / exposing port 5001 for backend

* environment variable for ngrok authtoken instead of hardcoding

* typo

* update readme

* delete ngrok container

* Delete .DS_Store

* run linter

---------
@leozhang14
Merge branch 'navbar' of https://github.com/uwblueprint/don-efficace …
752c335 
…into navbar
@pahu2353 @Ee-Hsin @leozhang14
basic table and filter created w/ chakra
f817942 
Co-authored-by: Jordan Kok Ee Hsin <[email protected]>
@pahu2353 @Ee-Hsin @leozhang14
basic table and filter created w/ chakra
3466ee0 
Co-authored-by: Jordan Kok Ee Hsin <[email protected]>
@pahu2353 @leozhang14
created table and filtering using chakra
10783b7
@leozhang14
adding navbar template
27ee212
@leozhang14
installing Tailwind + updating Navbar template to be closer to Figma …
c8684bf 
…design + routing for Navbar (temp)
@leozhang14
uncommenting subLabel - please note .eslintrc.js rules ignoring requi…
d7afe8c 
…re-default-props, no-unused-prop-types, and react-in-jsx-scope
@leozhang14
center navbar components + improve spacing + rename template names to…
3099f29 
… don efficace specific fields
@leozhang14
Navbar linking/routing + Updating navbar to be like Figma design
108fead
@leozhang14
fixed navbar spacing and link color - reorganized/simplified dashboar…
4553839 
…d/donations/account stack
@leozhang14
adding comments + changing Don Efficace. text to a link to homepage
308ce95
@aayushh-patell
Create Total Value Donated Component
f73c5e4 
This is just the front-end component. We still have to fetch data from the database.
@aayushh-patell
prisma.schema issue
8ebaff0
@aayushh-patell
Insert dummy data
59127b2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@victoryqwei victoryqwei Awaiting requested review from victoryqwei

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.
3 participants
@aayushh-patell @pahu2353 @leozhang14