Problem/Motivation

Currently, the required CKEditor library has to be manually downloaded and stored in the libraries folder for the module to work. We should add composer.json which will ensure library is downloaded when module is installed and also add instructions if user wants to use project's composer.json file for this.

Steps to reproduce

Install the module via composer.

Proposed resolution

Add composer.json file and instructions in the README.

Remaining tasks

NA

User interface changes

NA

API changes

NA

Data model changes

Issue fork tweetext-3256101

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:

Comments

gaurav.kapoor created an issue. See original summary.

anagomes’s picture

Assigned: Unassigned » anagomes

anagomes’s picture

Assigned: anagomes » Unassigned
Status: Active » Needs review

Hi @gaurav.kapoor. I added a composer.json file with the library as a requirement and a small comment on the README.txt, I hope everything is ok and according to what you expected. If you wish to alter or fix something, please let me know.

gaurav.kapoor’s picture

Great work @anagomes. The MR looks good. Can you set commit message to 'Issue #3256101 by anagomes: Add composer.json file and also ensure required libraries are installed using it'. Thanks.

gaurav.kapoor’s picture

Status: Needs review » Needs work

  • gaurav.kapoor committed b4e0f84 on 8.x-1.x
    Issue #3256101 by anagomes: Add composer.json file and also ensure...

  • gaurav.kapoor committed cc5958a on 8.x-1.x
    Issue #3256101 by anagomes, gaurav.kapoor: Add composer.json file and...
gaurav.kapoor’s picture

I researched a little bit and noticed that there is a https://github.com/drupal-ckeditor-libraries-group which hosts all the CKEditor plugins. I have requested them to add the plugin for this module as well.

@anagomes Thanks for the MR, most of the required code is available, and that and whatever that has to be modified will be available once we have this repository created.

  • gaurav.kapoor authored 5b2443b on 8.x-1.x
    Issue #3256101 by anagomes, gaurav.kapoor: Add composer.json file and...
gaurav.kapoor’s picture

Status: Needs work » Fixed

Added the composer.json file which will also install the required library. Tested by installing the latest development release via composer and noticed that library is getting installed now.

Status: Fixed » Closed (fixed)

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