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

Add onRamper webhook #725

Merged
merged 10 commits into from
Nov 29, 2022
Merged

Add onRamper webhook #725

merged 10 commits into from
Nov 29, 2022

Conversation

CarlosQ96
Copy link
Collaborator

@CarlosQ96 CarlosQ96 commented Nov 16, 2022

Onramper webhook. This flow occurs after the donation is done on the widget of onramper in the frontend.

Related to https://github.com/giveth/impact-graph#workspaces/all-devs-61644589b92edf0016f6999c/board

Documentation: https://docs.onramper.com/API-Reference/#webhooks

Tested with real data too. But I reproduced the same environment with the test key. So it's same functionality.

@CarlosQ96 CarlosQ96 changed the title Add onRamper webhook (wip) Add onRamper webhook Nov 29, 2022
@CarlosQ96 CarlosQ96 marked this pull request as ready for review November 29, 2022 08:45
Copy link
Collaborator

@mohammadranjbarz mohammadranjbarz left a comment

Choose a reason for hiding this comment

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

Thanks @CarlosQ96 for your clean code and refactoring udateDonation price peace of code, LGTM

PS: I just changed some little things and comments

@mohammadranjbarz mohammadranjbarz merged commit b0b2817 into staging Nov 29, 2022
@aminlatifi aminlatifi deleted the f_add_fiat_donations_webhook branch January 25, 2023 09:44
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