🇫🇷France @nicolasgraph

Posting rerolled patch.

I get the mentioned exception with the last version in date. I do not see any of patch related change in the module source code ; $context is still in use.
I cannot reopen the issue ; should I create a new one?

Merging 5.x introduced errors ; waiting #3428466 📌 Automated Drupal 11 compatibility fixes for commerce_payplug Needs review to land before to continue.

nicolasgraph → created an issue.

nicolasgraph → changed the visibility of the branch project-update-bot-only to hidden.

nicolasgraph → changed the visibility of the branch 5.x to hidden.

The previous commit added a wrong dependency declaration on commerce_payment:commerce_payment (>=3.0) instead of commerce:commerce_payment (>=3.0). Also, I think we should keep the D10.3+ support to stick to commerce versions support and make updates smooth. I personnaly uses commerce 3 on a D10.4 install.

nicolasgraph → created an issue.

Thanks for your reply @dom.
Unfortunatly I was not able to reproduce the bug in test mode.
In test mode, I'm not redirected to the site on payment failure, while in live mode I am.
The 5.x branch do not fix the issue bescause onReturn() is still empty and that's the method called when Payplug redirect to the site in my case.
Also I didn't try the v3 but I suspect #3295358 💬 Payments which are not completed, still complete the order. Closed: cannot reproduce to be a duplicate.

I confirm the current MR state do fix my issue in live mode.
Still needs attention and ideally, some tests.

Ok, I get it, in my failing manual test case, onNotify() is not called, the payment is managed through onReturn() which do... nothing.
While the payment entity creation can be managed onNotify() only, the remote payment validation must be processed onReturn() too to throw the DeclineException in case of failure.

nicolasgraph → changed the visibility of the branch 4.x to hidden.

Manual tests failed. The order is still placed.

Tests are now running, but failing, on d10.

This would need tests.
Waiting #3515036 📌 Fix tests Active to be fixed.

nicolasgraph → created an issue.

nicolasgraph → created an issue.

There is a client exception on the settings page after the Key module install; some more work needs to be done to make the install/uninstall as smooth as possible.

nicolasgraph → created an issue.

Completing the issue description and fix strategy.

nicolasgraph → created an issue.

nicolasgraph → created an issue.

nicolasgraph → created an issue.

Extends OrderItemSkuKernelTestBase in test once #3503345 📌 Explore different ways/moments to set the SKU Active is merged.

I reopen this issue to add a setting allowing to use a custom storing strategy.

nicolasgraph → created an issue.

Let's add tests later on and continue with #3505136 ✨ Sync order item SKU on purchased entity update Active .

nicolasgraph → created an issue.

On purchasable entity deletion the SKU storage should be managed via a queue.

I made a mistake in attributes ; fixing that…

This issue try to fix the Commerce 3 compatibility only.
Some more work may be needed to support D11.

nicolasgraph → created an issue.

nicolasgraph → created an issue.

I just created a light contrib module for this feature here → . I would just like to ensure that using "sku" as the trait field is ok ? I wouldn't like to conflict with a future implementation of this field.

An approach to this issue could be to introduce an entity trait for order items to choose to store the SKU if needed + a dedicated order processor. It is what I use for now ; I can help with a patch or a light contrib module.

Commited, thanks!

The balance is not related to the placed order status, it is related to the amount actually defined as paid by Alma. If no payment have already been processed for an order, the balance is 0. However, a queue worker should update the payments balance/status during the cron. Please check your logs and remote payments status or provide more informations to help me reproduce the bug.

Committed, thanks!

Patch #5 won't apply, it is an interdiff.
I created a MR from patch #4 including the suggested change of the interdiff.

@grevil, not sure which of your MR is the one you're woking on but I think you should keep the change I made to ReportsListBuilder.php in the MR#18. See #3342975 📌 Deprecate Url::toRenderArray() and Url::renderAccess() Fixed + #3406432 📌 Remove calls of deprecated toRenderArray() method Fixed .

Thank you all!

Thank you @guillaume-samtmann ; I think we could add an information message to the settings form if key is not installed to suggest its use.

Thanks @valthebald for this MR!

I pushed and revert a commit in favour of this comments:

1. In ai_translate.install, is ai_translate_requirements() still required? I guess not.
2. In ai_translate.module, _ai_translate_check_default_provider_and_model() seems also outdated to me.
3. In AiTranslateForm.php, line 114, $default_model = $this->providerManager->getSimpleDefaultProviderOptions('chat'); should not use the chat provider type anymore.

nicolasgraph → made their first commit to this issue’s fork.

Closing in favour of #3478721 📌 Discuss: Unify translate operations over ai Active which does fix this issue and adds many other improvements.

nicolasgraph → created an issue.

\Drupal call removed, local tests succeeded.

By pass review for this task.

nicolasgraph → created an issue.

The \Drupal call in the Cron class is documented as required. Tests are needed before to correct it.

nicolasgraph → created an issue.

nicolasgraph → created an issue.

nicolasgraph → created an issue.

Unwanted changes reverted and names added for width, length and height.

It's ok for me, I'll implement this hook ; we can revert my changes and explore manipulating the generated JSON-LD to extract the value from 'size' in the more specific properties later if needed.

Your approach makes sense. However, in the Commerce Shipping → case, a shippable product variation will have a dimension field, AND it could also have an attribute field for the size which could be XL for example. Do your think that this particular case should be handled by schemadotorg_commerce → ?