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

feat(payment_providers): Add Cashfree #2767

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

Conversation

vincent-pochet
Copy link
Collaborator

Roadmap Task

👉 https://getlago.canny.io/feature-requests/p/payment-provider-add-integration-for-cashfree-payments

Context

This PR replaces #2545 with a rebase of main to make sures that all checks are passing.

Cashfree Payments is India's leading payment gateway, and with Stripe halting the onboarding of new merchants in India due to regulatory constraints, businesses are left with limited alternatives. Ensuring that Cashfree Payments is available to users is essential, as it will expand Lago's accessibility and appeal to Indian companies, broadening its reach within this crucial market.

We at Pyrite Cloud have begun developing the integration with Cashfree Payments. This will help expand our platform's accessibility, especially for Indian businesses, by providing them with a reliable payment gateway as an alternative to Stripe.

Description

Added Cashfree Payment Provider concerning existing Payment Providers. Still Work in Progress.

@vincent-pochet vincent-pochet changed the title Add Cashfree Payment Provider feat(payment_providers): Add Cashfree Oct 31, 2024
@AyushChothe
Copy link

AyushChothe commented Nov 13, 2024

@vincent-pochet, Could you please provide an update on the progress of this PR and let me know when it can be merged?
CC: @jdenquin

@vincent-pochet
Copy link
Collaborator Author

@vincent-pochet, Could you please provide an update on the progress of this PR and let me know when it can be merged? CC: @jdenquin

Hello @AyushChothe.
Sorry, we had a lot to deal with this week and were not able to perform tests on this PR. Since this is a big change on a critical aspect of Lago, we really want to make sure it behave as expected, we are planning to invest some time on it next week.
Sorry and thank you again for you patience.

@AyushChothe
Copy link

Ok, no worries! Thanks for the update.

@AyushChothe
Copy link

Hi @vincent-pochet, any more updates on this?

@AniketKale-coder
Copy link

AniketKale-coder commented Dec 1, 2024

@jdenquin @vincent-pochet, it has been a while since we have waited for this feature to be merged. Please provide any updates on this; we are stuck because of it.

@vincent-pochet
Copy link
Collaborator Author

Hello @AniketKale-coder
We are aware that it took a really long time on our side, we are deeply sorry about that. We have a lot to deal with on many aspect of the product.

I took some time today to improve the code to match some changes that were introduced since your initial Pull Request. We will conduct a QA in the coming days.
We also still have to figure how to deal with the "refund" (Credit Notes) part as it is not handled today

@vincent-pochet vincent-pochet force-pushed the cashfree-integration branch 2 times, most recently from 06bacfa to 07688fe Compare December 6, 2024 15:57
@AyushChothe
Copy link

Hi @vincent-pochet, could you please provide an update on this?

@vincent-pochet
Copy link
Collaborator Author

Hi @vincent-pochet, could you please provide an update on this?

Hi @AyushChothe,
We did a first round of QA, and we identified an issue that we have to fix. A 500 error was raised when we tried to generate the payment URL and we still have to investigate.

In parallel of that, a major issue was identified in the payment processing implementation leading to a risk of duplicated payments when the system is under heavy load.
The decision was then made to refactor some parts of this implementation to fix this.
We will jump back on the QA and release of Cashfree when everything will be fixed. We are hopping that it will be next week.

Sorry about this, we will keep you posted.

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.

4 participants