Problem/Motivation

Using Composer for installation is more and more becoming the standard. Yet, the libraries are still be uploaded the manual way using i.e. filezilla

Proposed resolution

create a composer.libraries.json file so the libraries can be uploaded using the wikimedia composer-merge-plugin

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork flexslider-3185456

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bart lambert created an issue. See original summary.

Pooja Ganjage’s picture

Hi,

Creating a patch for this issue.

Please review the patch.

Let me know if any suggestions.

Thanks.

Pooja Ganjage’s picture

Status: Active » Needs review
bart lambert’s picture

tested but got an error that download failed:

Pooja Ganjage’s picture

Updated patch.

Please review.

Thanks.

bart lambert’s picture

tested and working on php 7.4 & MySQL 5.7 D8.9.10

bart lambert’s picture

Composer is actually being update to version 2.0 which poses problems with the wikimedia/composer-merge-plugin:^1.4.1.

Persons using composer 2.0 have to use:

wikimedia/composer-merge-plugin:"dev-feature/composer-v2 as 1.5.0"

So in the composer.libraries.json file line 20:

"wikimedia/composer-merge-plugin": "^1.4.1" should be replaced by

"wikimedia/composer-merge-plugin": "dev-feature/composer-v2 as 1.5.0"

I don't know if this newer version is backwards compatible with the composer version 1

bart lambert’s picture

ANother Hickup!

Using composer-merge-plugin and the proposed composer.libraries.json file, the library is installed in the libraries-folder in the folder woocommerce.flexslider and actually it needs the foldername flexslider

bart lambert’s picture

Ruslan Piskarov made their first commit to this issue’s fork.

Ruslan Piskarov’s picture

Seams as @bart lambert added a json file instead of a patch. I have committed the fix.

Ruslan Piskarov’s picture

Title: composes.libraries.json » Add composes.libraries.json to flexslider project
bart lambert’s picture

thanks!

liquidcms’s picture

Composer can't be used to apply this patch, correct? Chicken/egg issue. So the only was to use this is if it gets committed?

Ruslan Piskarov’s picture

euk’s picture

Would be great to have this added to the D10 tagged version...

sleitner made their first commit to this issue’s fork.

sleitner’s picture

Version: 8.x-2.0 » 3.x-dev

Please review MR11 for 3.x

rossb89 made their first commit to this issue’s fork.

rossb89’s picture

I updated MR-11 with the latest changes in 3.x as it was a bit out of date and not mergeable.

I'm not quite sure why we have both MR-1 *and* MR-11 though?

Update - oh sure it's because MR-11 was done for 3.x but it's the same exact branch as MR-1 was for 2.x 😅

Ruslan Piskarov’s picture

Deleted comment. Sory.