https://drupalcommerce.org/blog/49669/installing-commerce-2x-without-com...

Having Ludwig integration would allow people to use the module without Composer.

Patch incoming.

CommentFileSizeAuthor
#3 2891683-3.patch2.01 KBbojanz
#2 2891683-2.patch1.52 KBbojanz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bojanz created an issue. See original summary.

bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Status: Active » Needs review
FileSize
2891683-2.patch1.52 KB

Also took the opportunity to fix composer.json (indentation, windows line endings)

bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
FileSize
2891683-3.patch2.01 KB

Previous patch was incomplete (ludwig.json missing the geoip2 dependencies)

  • bojanz committed b36b46d on 8.x-2.x 
    Issue #2891683 by bojanz: Add Ludwig integration
bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Status: Needs review » Fixed

Updated the versions and committed.

Status: Fixed » Closed (fixed)

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

devad’s picture

devad CreditAttribution: devad as a volunteer commented
Related issues: +#3174645: [LIST] Ludwig integrations in contrib modules
devad’s picture

devad CreditAttribution: devad as a volunteer commented
Title: Add Ludwig integration » GeoIP - Ludwig integration
devad’s picture

devad CreditAttribution: devad as a volunteer commented
Related issues: -#3174645: [LIST] Ludwig integrations in contrib modules

Undo. :)

devad’s picture

devad CreditAttribution: devad as a volunteer commented

New Ludwig 8.x-1.1 has added a full support to maxmind/web-service-common library.

If you are using Ludwig 8.x-1.0 still - an upgrade to Ludwig 8.x-1.1 is recommended.

Previous problem with multiple-resource PSR-4 libraries (such as maxmind/web-service-common) is solved by iterating through all the resources now and adding them as individual namespace records.