Closed (fixed)
Project:
Collapsiblock
Version:
4.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
25 Jul 2022 at 09:27 UTC
Updated:
8 May 2023 at 07:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
maxilein commentedComment #3
darvanenCan you expand on why you think this is a GDPR issue?
Comment #4
emptyvoid commentedExternal resource throwing a 503 error and blocking all requests.
This causes over 20 second hang to render every single screen on all drupal sites using this module.
https://unpkg.com/slide-element@2/dist/index.umd.js
Comment #5
emptyvoid commentedpatch against 4.x dev branch including external js to remove 20-30 error timeout and page draw holds.
Comment #7
darvanenI'm not comfortable with including the package directly in the module. I've taken a lead from the Webform module and kept the CDN as a fallback while allowing people to install the library in their project in an appropriate location. Thanks for your nudges to get this done.
Comment #8
chandu7929 commentedDue to this changes ExistingSiteJavascript test are failing, its seems that libraries is not available during test.
Note: I have not downloaded the lib locally.
Comment #9
darvanenThanks for calling that out, shouldn't be too difficult to solve, I might have time this weekend but I'm very happy for someone else to take a look at it too. I'm out or I would open a new ticket for this.
Comment #10
chandu7929 commentedCreated child issue for the same.