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

iDeal Integration (online payment provider) #1008

Closed
yksflip opened this issue Jun 12, 2023 · 5 comments
Closed

iDeal Integration (online payment provider) #1008

yksflip opened this issue Jun 12, 2023 · 5 comments
Labels
Milestone

Comments

@yksflip
Copy link
Member

yksflip commented Jun 12, 2023

Feature request by ardjan
https://forum.foodcoops.net/t/feature-request-adding-ideal-payment-method-to-upstream-version/625

already mentioned in another post is the iDeal integration for online payments with the Dutch banking system, also working in the Adam fork.

Already inside foodcoops-adam.

Looks like there's already a plugin in foodcoop-adam: foodcoop-adam@ef824ea

@yksflip yksflip added this to the 4.9 milestone Jun 12, 2023
@yksflip yksflip changed the title iDeal Integration iDeal Integration (online payment provider) Jun 12, 2023
@wvengen
Copy link
Member

wvengen commented Jun 15, 2023

The foodcoop-adam plugin currently allows members to transfer an amount from their bank (through iDeal) to their Foodsoft account. Members can choose the amount themselves. Foodcoops can use the option to prohibit ordering if a member's account balance is below a threshold value, to make sure people keep enough money on their Foodsoft account.

Please note it does not currently integrate with the ordering system. The feature in the Adam fork that allows one to order, pay, and only then have the member order go through, is not part of the iDeal plugin. So members need to think for themselves to top up their Foodsoft account balance. (This could be improved by adding some links/buttons/logic here and there, I suppose.)

@wvengen
Copy link
Member

wvengen commented Jun 15, 2023

Also, I would link to the in-tree plugin here: https://github.com/foodcoop-adam/foodsoft/tree/beta/lib/foodsoft_mollie
Just checked, this does not yet expose the functionality in Foodsoft yet, so it would make sense to add some deface overrides e.g. on the account balance page, and on the home page (e.g. if your balance is below zero).

@yksflip
Copy link
Member Author

yksflip commented Sep 14, 2023

ah great, the beta branch already uses mollie-api-ruby, some changes for the new api version are required, but it looks already very good. In foodsoft-adam you've added some attributes to the FinancialTransaction Model (like payment_method, payment_state etc.) to track the transaction status. Do you think this this still the right place for this attributes?
I'm a bit confused by the financial_transaction_type and financial_transaction_class , do they offer a abstraction for this sort of thing? Or is it rather a thing foodcoops use to categorize their transactions?

@wvengen
Copy link
Member

wvengen commented Sep 14, 2023

I think it can make sense to keep it with the FinancialTransaction model. If not, you could add an online payment model that mirrors the financial transactions which need to be tracked. (Btw, this tracking is important, because an online payment can succeed or fail, and Foodsoft may be notified later even after the user is offline again.)

The Financial transaction type and class are something introduced a while ago, and I think are mostly used by the Austrian foodcoops. The idea is that you can e.g. pay membership fee and show it separately in Foodsoft, or 'invest' and have it tracked but not used to pay member orders. I think that, just when entering a payment manually, you may need to specify the transaction type when doing an online payment (this is not implemented in the plugin yet, as the transaction types/classes were added later to Foodsoft). I think the financial transaction type/class system is in bad need of documentation (preferably with thelp of the people who actively use it).

If you allow specifying the financial transaction type as a query parameter too, then a deface override could be made to link from e.g. the dashboard to the payment page to pay the amount due (prefilled using the query parameters). This is currently also missing in the payment plugin, I think.

@yksflip
Copy link
Member Author

yksflip commented Sep 15, 2023

thanks for your advise willem, I'll give it a shot :)

yksflip added a commit that referenced this issue Apr 8, 2024
…1008 , PR #1031)

Co-authored-by: wvengen <willem@thequestionmark.org>
yksflip added a commit that referenced this issue Apr 8, 2024
Co-authored-by: wvengen <willem@thequestionmark.org>
yksflip added a commit that referenced this issue Apr 8, 2024
…1008 , PR #1031)

Co-authored-by: wvengen <willem@thequestionmark.org>
yksflip added a commit that referenced this issue Apr 8, 2024
Co-authored-by: wvengen <willem@thequestionmark.org>
@yksflip yksflip closed this as completed Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

No branches or pull requests

2 participants