New version of the module

Hi Remco,

Thank you for raising this. I believe https://www.drupal.org/u/matroskeen → has started this work as part of https://www.drupal.org/project/shopify/issues/3374457 📌 Create a new class Shopify with getClient() method Fixed

I'm not sure how far is he in the implementation, but he created the 2.0.x branch with his changes.

Can you try contacting him directly? Either via https://www.drupal.org/user/3426249/contact → or Drupal Slack.

Hi Remco,

You're very welcome to collaborate and submit your code suggestions to dedicated issues. Feel free to use 2.0.x branch, that I created but haven't had time to make something real.

Unfortunately, I'm not available to actively collaborate and review the code, since I'm in the army now.
You can try involving other community members to review the code. If you add some automated tests to cover new changes, that would be even better.

Let me know if you need anything else to start working.
Thanks!

Thank you for the response guys! Appreciate it, will make a MR / PR this weekend with the changes needed. Ill also look into automated tests.

With friendly greetings

Remco

ccjjmartin → closed merge request !35

ccjjmartin → changed the visibility of the branch 3414025-d10-updates to hidden.

ccjjmartin → changed the visibility of the branch 3414025-d10-updates to active.

@ccjjmartin I think we're trying to move away from "donutdan4114/shopify" in favor of Shopify's official API.

Really happy to hear that you are picking up this project! I have a site still on Drupal 9 since this is a blocker to upgrade. Is this something we can expect in the near future or will this be several months away still?

You can expect this in the near future! Will be working to get this up and running in the summer.

@noregrebt & anyone else not able to update to D10:

After initially installing the module with composer in D9, I:
* moved the module to the /web/modules/custom folder
* applied this MR as a patch https://git.drupalcode.org/project/shopify/-/merge_requests/33/diffs - you can download it as a patch from the 'code' dropdown button top right
* removed the module from composer
* added the module to my git repository

This allowed me to update to D10 & it's been about 8 months, have had no issues.

An installation method with Composer as an alternative to the method described in #15

Step 1:

composer require mglaman/composer-drupal-lenient

Step 2:

composer config --merge --json extra.drupal-lenient.allowed-list '["drupal/shopify"]'

Step 3: past patch( currently MR36) in composer.json

"drupal/shopify":{
                "New version of the module-issues-3414025":"https://git.drupalcode.org/project/shopify/-/merge_requests/36.patch"
            }

Step 4:

composer require 'drupal/shopify:2.0.x-dev@dev'

MR!36 tested with Drupal 10.3

Error:

Error: Call to a member function getResources() on null in Drupal\shopify\Form\ShopifyWebhooksAdminForm->buildForm() (line 38 of /var/www/html/d103test/web/modules/contrib/shopify/src/Form/ShopifyWebhooksAdminForm.php).

#0 [internal function]: Drupal\shopify\Form\ShopifyWebhooksAdminForm->buildForm()
#1 /var/www/html/d103test/web/core/lib/Drupal/Core/Form/FormBuilder.php(536): call_user_func_array()
#2 /var/www/html/d103test/web/core/lib/Drupal/Core/Form/FormBuilder.php(284): Drupal\Core\Form\FormBuilder->retrieveForm()
#3 /var/www/html/d103test/web/core/lib/Drupal/Core/Controller/FormController.php(73): Drupal\Core\Form\FormBuilder->buildForm()
#4 [internal function]: Drupal\Core\Controller\FormController->getContentResult()
#5 /var/www/html/d103test/web/core/lib/Drupal/Core/EventSubscriber/EarlyRenderingControllerWrapperSubscriber.php(123): call_user_func_array()
#6 /var/www/html/d103test/web/core/lib/Drupal/Core/Render/Renderer.php(638): Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->Drupal\Core\EventSubscriber\{closure}()
#7 /var/www/html/d103test/web/core/lib/Drupal/Core/EventSubscriber/EarlyRenderingControllerWrapperSubscriber.php(121): Drupal\Core\Render\Renderer->executeInRenderContext()
#8 /var/www/html/d103test/web/core/lib/Drupal/Core/EventSubscriber/EarlyRenderingControllerWrapperSubscriber.php(97): Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->wrapControllerExecutionInRenderContext()
#9 /var/www/html/d103test/vendor/symfony/http-kernel/HttpKernel.php(181): Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->Drupal\Core\EventSubscriber\{closure}()
#10 /var/www/html/d103test/vendor/symfony/http-kernel/HttpKernel.php(76): Symfony\Component\HttpKernel\HttpKernel->handleRaw()
#11 /var/www/html/d103test/web/core/lib/Drupal/Core/StackMiddleware/Session.php(53): Symfony\Component\HttpKernel\HttpKernel->handle()
#12 /var/www/html/d103test/web/core/lib/Drupal/Core/StackMiddleware/KernelPreHandle.php(48): Drupal\Core\StackMiddleware\Session->handle()
#13 /var/www/html/d103test/web/core/lib/Drupal/Core/StackMiddleware/ContentLength.php(28): Drupal\Core\StackMiddleware\KernelPreHandle->handle()
#14 /var/www/html/d103test/web/core/modules/big_pipe/src/StackMiddleware/ContentLength.php(32): Drupal\Core\StackMiddleware\ContentLength->handle()
#15 /var/www/html/d103test/web/core/modules/page_cache/src/StackMiddleware/PageCache.php(106): Drupal\big_pipe\StackMiddleware\ContentLength->handle()
#16 /var/www/html/d103test/web/core/modules/page_cache/src/StackMiddleware/PageCache.php(85): Drupal\page_cache\StackMiddleware\PageCache->pass()
#17 /var/www/html/d103test/web/core/lib/Drupal/Core/StackMiddleware/ReverseProxyMiddleware.php(48): Drupal\page_cache\StackMiddleware\PageCache->handle()
#18 /var/www/html/d103test/web/core/lib/Drupal/Core/StackMiddleware/NegotiationMiddleware.php(51): Drupal\Core\StackMiddleware\ReverseProxyMiddleware->handle()
#19 /var/www/html/d103test/web/core/lib/Drupal/Core/StackMiddleware/AjaxPageState.php(36): Drupal\Core\StackMiddleware\NegotiationMiddleware->handle()
#20 /var/www/html/d103test/web/core/lib/Drupal/Core/StackMiddleware/StackedHttpKernel.php(51): Drupal\Core\StackMiddleware\AjaxPageState->handle()
#21 /var/www/html/d103test/web/core/lib/Drupal/Core/DrupalKernel.php(741): Drupal\Core\StackMiddleware\StackedHttpKernel->handle()
#22 /var/www/html/d103test/web/index.php(19): Drupal\Core\DrupalKernel->handle()
#23 {main}