Hi
I would like to throw the idea to add Ludwig module support to this readily awesome barcodes module so that it can become more awesome.
A lot of us are using cpanel with very limited access to the server, a lot of the times my clients already have their own hosting server so it is troublesome to have to configure this barcode module every time without access to composer.
Therefore, it would be very great if this awesome barcodes module could also use ludwig in addition to composer.
Thank you.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | barcodes-ludwig-integration-3160901-4.patch | 462 bytes | devad |
Comments
Comment #2
sanduhrsComment #3
devad commentedComment #4
devad commentedHi @sanduhrs and other maintainers of Barcodes module.
I am Ludwig module co-maintainer.
I would like to support idea that you add Ludwig integration into Barcodes module by providing
ludwig.jsonfile inside your module's root folder in all future Barcodes module 2.x.x releases.This would make the life of users not friendly with Composer much easier. :)
Here is the full list of (currently) required libraries for Barcodes module installation from composer report:
So,
tecnickcom/tc-lib-colorlibrary is a dependency oftecnickcom/tc-lib-barcode, so both these libraries have to be added toludwig.jsonfile in order for Barcodes module to work nicely with Ludwig.Maintenance would not be a big issue in this case. Once new Barcodes module release is out and if library dependencies has changed -
ludwig.jsonfile should be updated to new library requirements by Barcodes module maintainers just like composer.json file is updated.Here is the patch. I created it fo 2.0.x-dev, but it should fit for 8.x-1.x-dev also.
It's pity that Barcode module tests are not enabled. :)
Comment #5
devad commentedComment #6
devad commentedUndo. :)
Comment #7
devad commentedComment #10
sanduhrsCommitted, thanks for the patch ;)
Comment #11
devad commentedWelcome. Thanks for commit and new releases!