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

fix: restyle donation buttons, add omprakash #210

Draft
wants to merge 1 commit into
base: saga
Choose a base branch
from

Conversation

jimthedev
Copy link

Note: opened as a draft PR since I have some questions I need answered.

What changed?

#89

How will this change be visible?

Donation components will change. I did add some questions / comments to the project task

How can you test this change?

  • Automated tests
  • Manual tests (describe)

I will probably just visual test these. None of them have any props at the moment. I don't really know how they intend to be reused yet. We could add some props if that would be helpful but it seems a bit like we just want to know they can be reused. Happy to add these buttons other places in this PR.

@jtfairbank
Copy link
Contributor

@jimthedev is this one still in draft state or ready for review? Agree that visual test for these is fine for now!

@jimthedev
Copy link
Author

@jtfairbank I got ahead of myself so this is still a draft for now. I did leave some questions on #89 that will probably need to be looked over for me to proceed.

I probably need to get connected in more to proceed. I expect being at some coworking sessions will help.

@jtfairbank
Copy link
Contributor

Thanks for flagging that @jimthedev! Replied to your questions and hope to see you at some of the coworks this week. 😄

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.

2 participants