ecommerce_integrations icon indicating copy to clipboard operation
ecommerce_integrations copied to clipboard

Feature Request: Map order Payment Gateway to Payment Account

Open meaziz opened this issue 2 years ago • 7 comments

We stumbled upon a case where we wants to have the payment entry posted to different accounts based on the payment gateway used in shopify..

for example

Payment Gateway Account Name
Cash on delivery 11202 Fullfilment Cash
Credit Card 11203 Credit Card.

or should it map to mode of payment which maps to a cash or bank account.

I would recommend keeping it simple to a bank account..

We can contribute this if we agreed on the specs.

meaziz avatar Oct 22 '22 08:10 meaziz

Hello, giving option for gateway -> account seems most flexible for future use cases too. (similar to location->warehouse and tax accounts)

This seems fine. Tag me when you open PR :)

ankush avatar Nov 08 '22 13:11 ankush

@ankush tagging you as per your request not sure why haven't we opened the pull request earlier 🥲

meaziz avatar Jun 08 '23 17:06 meaziz

Will this feature be added soon? Our Bank account don't match in ERPNext because Payment Gateway is always the same.

azurre931 avatar Nov 29 '23 17:11 azurre931

We have it in production for 4 months now. It needs minor cleanup before we open a new pull request for it.

And add the proper documentation.

On Wed, Nov 29, 2023 at 7:30 PM azurre931 @.***> wrote:

Will this feature be added soon? Our Bank account don't match in ERPNext because Payment Gateway is always the same.

— Reply to this email directly, view it on GitHub https://github.com/frappe/ecommerce_integrations/issues/213#issuecomment-1832391730, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYHTZNGA7LUPZ7SZHIUCBTYG5WM5AVCNFSM6AAAAAARLYFIHOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZSGM4TCNZTGA . You are receiving this because you authored the thread.Message ID: @.***>

meaziz avatar Nov 29 '23 17:11 meaziz

Do we have any news about this feature? Thank you.

azurre931 avatar May 27 '24 14:05 azurre931

It is working on a fork maybe we should refine it and open a PR for it.

meaziz avatar May 27 '24 19:05 meaziz

Can you open a PR for it? Thank you!

azurre931 avatar Sep 19 '24 17:09 azurre931