Created on 22 October 2023, about 1 year ago
Updated 16 February 2024, 9 months ago

Problem/Motivation

Using in D10

- using drupal/once library instead of jquery/once
- fix version file for d10 compatibility

Feature request
Status

Needs review

Version

1.0

Component

Code

Created by

🇩🇪Germany spuky

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

Comments & Activities

  • Issue created by @spuky
  • Assigned to mohd sahzad
  • Please ask your managers to make some donations to support the development of the module.

  • Issue was unassigned.
  • 🇮🇳India mohd sahzad

    I have updated this issue in attached patched patche1 please review.

  • Status changed to Needs review about 1 year ago
  • 🇩🇪Germany spuky

    @Mohd Sahzad here is my review of your Patch

    your patch is missing the JS Code to to use drupal/once instead of jquery.once
    see my patch that came wit the Feature request..

    also you are stating that the module works with D8 but list drupal/once as a library which does not exits for D8
    also D8 is EOL since November 2021 so nobody needs encouragement to stay on 8

    Looking forward for the review of mine ;-)

    @wombatbuddy my patch is already working... and tested

    on my D10 Site...
    have it in my composer Json.. using Drupal Lenient Composer Endpoint

    so i was just thinking about other users...
    or you as a maintainer to just have to merge the request...

  • First commit to issue fork.
Production build 0.71.5 2024