Please add Drupal 10 compatibility

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

The changes to line 57 of js/commerce_unzer.form.js seem wrong. Do they work for you?

Hi, sorry for my mistake. I raised the PR without checking. But now I have fixed this mistake.

Please review this new MR.

The patch updates the PHP version requirement to >= 8.1 although the Unzerdev SDK still supports 7.4.

Since Drupal 9 is no longer supported now ( https://www.drupal.org/docs/understanding-drupal/drupal-9-release-date-a... → ) and Drupal 10 requires at least PHP 8.1 ( https://www.drupal.org/docs/getting-started/system-requirements/php-requ... → ) I think we can accept that.

Here's the change record for the JavaScript change: https://www.drupal.org/node/3158256 →

@gruberroland : I committed the patch that removes the dependency on jQuery once.

• Are you aware of any remaining issues with respect to D10 compatibility?
  Other than packaging a release, I mean.
• Are you using the onsite payment gateway? That part is known to have worked at least at some point but we do not use it in our project so it sees little testing.

I can confirm that the current DEV version works with Drupal 10.1.6 with offsite payment gateway.
The onsite gateway seems to work, too. My test order was successful. But I do not use this in production as it does not support e.g. SEPA direct debit.

I created an issue some days ago about code quality improvements and modernization:
https://www.drupal.org/project/commerce_unzer/issues/3403407 📌 Bring code up to snuff Fixed

I made the changes I had in mind and will soon create a new alpha release.

cspitzlay → closed merge request !2

Hello @cspitzlay, taking into account that the Merge request was closed 3 months ago, it could be said that this issue is already solved and therefore this should be marked as "Fixed", right? os is there something else undone? Thanks!

Well, we are still at an alpha version and not a regular release.
But then again: D11 is near so keeping this open may not make much sense.