D10: Update recommendation for ACE Editor (drupal/ace_editor)

Created on 24 January 2024, 11 months ago
Updated 9 February 2024, 11 months ago

Problem/Motivation

This module was vetted in our D9 curated.json and was deemed obsolete due to requiring manual installation.

The reasoning behind this was due to this underling issue, https://www.drupal.org/project/ace_editor/issues/3022330 πŸ› composer installation fail Fixed , which now appears Closed/Fixed.

On the project page for Ace Editor β†’ there is a beta release which should have D10 compatibility. This should be re-vetted to determine if manual installation is still necessary or if things have been resolved.

See 3416782 for more details πŸ“Œ Review list of obsolete modules to determine if status is unchanged Active

Steps to reproduce

n/a

Proposed resolution

  1. Re-vet the module to determine if there is a viable path to D10

Remaining tasks

  1. Install on fresh D7 site and set up
  2. Install on fresh D10 site and set up
  3. If both install and work ok, add to curated.json and rebuild recommendations.json + D10 project with ACE Editor + AMA

User interface changes

n/a

API changes

n/a

Data model changes

n/a

πŸ“Œ Task
Status

Fixed

Version

1.9

Component

Recommendations

Created by

πŸ‡ΊπŸ‡ΈUnited States dan612 Portland, Maine

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @dan612
  • πŸ‡ΊπŸ‡ΈUnited States dan612 Portland, Maine

    If you try to include the D10 compatible version via AM:A, the following error is thrown when installing a new project:

    Your requirements could not be resolved to an installable set of packages.
    
      Problem 1
        - Root composer.json requires drupal/ace_editor 2.0.0-beta2 -> satisfiable by drupal/ace_editor[2.0.0-beta2].
        - drupal/ace_editor 2.0.0-beta2 requires npm-asset/ace-builds ~1.0 -> could not be found in any version, there may be a typo in the package name.
    
    Potential causes:
     - A typo in the package name
     - The package is not available in a stable-enough version according to your minimum-stability setting
       see <https://getcomposer.org/doc/04-schema.md#minimum-stability> for more details.
     - It's a private package and you forgot to add a custom repository to find it
    

    This is because we have to alter the project composer.json file to allow the downloading of the library, see this issue for more details πŸ“Œ Update README.md Composer steps, and format to Markdown Needs review . I was able to get the ACE Editor up and running on my D10 site after following the README updates in that issue, however this still seems like it needs to be done manually for the time being unless we can somehow work in AM:A some specialized handling of composer.json generation should this module (or others like it) be present.

    This also needs to be tested with data that comes from Drupal 7 to ensure it works the same on Drupal 10 with identical underlying data after a migration.

  • πŸ‡ΊπŸ‡ΈUnited States dan612 Portland, Maine

    Updated in the issue fork attached to this issue πŸ“Œ Review list of obsolete modules to determine if status is unchanged Active . Closing.

  • Status changed to Fixed 11 months ago
  • πŸ‡ΊπŸ‡ΈUnited States dan612 Portland, Maine
  • Issue was unassigned.
  • πŸ‡§πŸ‡ͺBelgium wim leers Ghent πŸ‡§πŸ‡ͺπŸ‡ͺπŸ‡Ί
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024