Hi!
We would like to ensure that all Commerce 2.x payment gateways have Ludwig integration, to ensure that they can be installed without Composer.
You can find out more about Ludwig from this blog post: https://drupalcommerce.org/blog/49669/installing-commerce-2x-without-com...
And you can find a sample Ludwig integration here: http://cgit.drupalcode.org/commerce_ingenico/commit/?id=364f5ea

Notes:

- If your SDK has a dependency of its own in its composer.json, you must list that in the ludwig.json file as well

- You do not need to list dependencies that are included in Drupal core (such as Symfony)

If you have any doubts, you can post a patch to this issue first, and then someone from the Commerce Guys team will review.

Thank you!

CommentFileSizeAuthor
#2 add_ludwig_interaction--2892644-2.patch317 bytesdom.

Comments

krcha created an issue. See original summary.

dom.’s picture

Status: Active » Needs review
StatusFileSize
new317 bytes

Patch attached.

  • Dom. committed 7da05b8 on 8.x-1.x
    Issue #2892644 by Dom.: Add Ludwig integration
    
dom.’s picture

Status: Needs review » Fixed
bojanz’s picture

Status: Fixed » Needs work

You should also add a hook_requirements to your .install file like in the linked example.
That will prevent a crash for people who ignored all instructions and tried to install the module without the library.

dom.’s picture

Status: Needs work » Postponed (maintainer needs more info)

There is already a hook_requirements in the module. Does it miss something ?
http://cgit.drupalcode.org/commerce_payplug/tree/commerce_payplug.install

bojanz’s picture

Status: Postponed (maintainer needs more info) » Fixed

Sorry, I missed that because it wasn't in the commit.

Thank you for your efforts!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

devad’s picture

Title: Add Ludwig integration » Commerce Payplug - Add Ludwig integration