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

Is it possible to stop changing the Klarna states? #296

Closed
jacob-v-dam opened this issue Feb 19, 2021 · 8 comments · Fixed by #318
Closed

Is it possible to stop changing the Klarna states? #296

jacob-v-dam opened this issue Feb 19, 2021 · 8 comments · Fixed by #318

Comments

@jacob-v-dam
Copy link

Expected behavior

When i change a Klarna order it should not set it's own state.

Actual behavior

When we change the state to shipped the order get's the Klarna shipped state:
image

Steps to reproduce the behavior

Get an authorized Klarna payment. Wait a few minutes, change the order state to shipped. The state will change to Klarna shipped directly.

Environment

  • PrestaShop version: 1.7.7.1
  • Mollie module version: 4.2.2
  • PHP version: 7.4
@Saimis777
Copy link
Contributor

Hey @jacob-v-dam ,

Thank you for your message!

I will clarify this issue with the team and get back to you as soon as possible!

--
Best regards,
Invertus Support team

@Saimis777
Copy link
Contributor

Hello @jacob-v-dam ,

I've got a clarification that this logic at the moment will stay as it is as we had to add custom klarna statuses because of the klarna logic where invoice can be created when authorized or shipped.

I will keep you posted if this will be edited or changed in any way, but at the moment that is how it works.

Maybe you have any other questions in the meantime?

--
Best regards,
Invertus Support team

@AmandaMollie
Copy link

@jacob-v-dam - As @Saimis777 mentioned we've created these statuses in order to be able to delay the sending of the invoice until the Klarna order has been shipped.

As we look into designing a solution that does not create an additional status, could you let me know if you also have the need for the delayed invoice? To put it simply, would option 2 work for your purposes?

Options:

  1. custom "shipped" status = delayed invoice trigger
  2. standard "shipped" status = no delayed invoice trigger

@Saimis777
Copy link
Contributor

Hello @jacob-v-dam ,

Could you clarify if Amanda's 2nd suggested option would work for you? Or maybe you have another suggestion?

Let us know!

--
Best regards,
Invertus Support team

@Saimis777
Copy link
Contributor

Hello @jacob-v-dam ,

Could you clarify if you've managed to check the suggested options and if they would work for you?

Looking forward to your reply!

--
Best regards,
Invertus Support team

@Saimis777
Copy link
Contributor

Hello @jacob-v-dam ,

Due to inactivity I will close this issue for now.

Let me know once you'll check the options or will have any ideas in mind!

Have a nice day!

--
Best regards,
Invertus Support team

@Maz-Invertus
Copy link
Contributor

Our team is working on the solution to add additional option when to create an invoice:

When to create the invoice?

Dropdown values:

Default
Authorized
Shipped

Description:

When do you want the invoice for Klarna Payments?
Default: Invoice creation is based on order settings > statuses. Custom status is not created.
On Authorize: Create a full invoice when the order is authorized. Custom status is created "Klarna payment authorized".
On Shipment: Create a full invoice when the order is shipped. Custom status is created "Klarna payment shipped".

@Saimis777
Copy link
Contributor

Hello,

A new version of Mollie was just released, with a solution included for you, you can download the latest version here - https://github.com/mollie/PrestaShop/releases/tag/4.2.4

We are closing this issue for now, if the issue is still there be sure to let us know, we'll reopen the issue!

And be sure to provide us feedback regarding the latest release!

--
Best regards,
Invertus Support team

margud added a commit that referenced this issue Jun 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants