pretix / pretix-mollie

Mollie payments for pretix
Other
3 stars 9 forks source link

Feature-request: test-payments #20

Closed toontoet closed 1 year ago

toontoet commented 1 year ago

The current PreTix plugin for Mollie doesn't support test-payments when the shop is in test-mode. A message is shown that Mollie does not support test-transactions. However, currently Mollie does support test-transaction. They give out test API-key's that are prefixed by test_ instead of live_. The Mollie settings in PreTix don't accept an API-key that doesn't start with live_ though.

Being able to test transactions using an test-key would be awesome someday.

pc-coholic commented 1 year ago

Looking at the source, the plugin should already work like that...

For non-connect usage (where you enter an API-key for every single event), the form should accept live_ and test_-keys: https://github.com/pretix/pretix-mollie/blob/a6a0014c670fbd112d8128ad72e401790ad87fe9/pretix_mollie/payment.py#L144-L154

As for the message: If the API-key starts with test_ an appropriate message should be displayed: https://github.com/pretix/pretix-mollie/blob/a6a0014c670fbd112d8128ad72e401790ad87fe9/pretix_mollie/payment.py#L105-L113

If you are using a live-key right now, the message that test-transactions are not supported is expected.

Could you please verify/confirm, that you are not using the "Mollie Connect"-path (no Client-ID/Client-Secret in the global settings) and possibly provide some screenshots, where the test_-Key is not accepted? I just tried it in my system and it works as intended...

toontoet commented 1 year ago

That's strange. It didn't accept the test_ key when I tried before. But is does now. Maybe a space was prefixed when copy-pasting ...

The warning on the show still shows the message about the test-mode that is not supported and real money could be charged. When I finished the order test-mode did work al expected. I guess it might be a translation issue...

Screenshot 2023-04-12 at 12 53 44