Adobe-Consulting-Services / acs-aem-commons

http://adobe-consulting-services.github.io/acs-aem-commons/
Apache License 2.0
453 stars 598 forks source link

Versioned ClientLibs affecting Preview Package generation (e.g. SDL Translation Preview ) #2338

Closed tlocwin closed 4 years ago

tlocwin commented 4 years ago

Required Information

Expected Behavior

<link rel="stylesheet" href="../../../../var/designs/about-us_1/jquery-ui.css" type="text/css">
<link rel="stylesheet" href="../../../../var/designs/about-us_1/site.css" type="text/css">
<link rel="stylesheet" href="../../../../var/designs/about-us_1/clientlib-base.css" type="text/css">
<link rel="stylesheet" href="../../../../var/designs/about-us_1/default.css" type="text/css”>

Preview OK Screenshot 2020-06-23 at 14 36 21

Preview NOK Screenshot 2020-06-23 at 13 46 06

Actual Behavior

<link rel="stylesheet" href="/etc.clientlibs/clientlibs/granite/jquery-ui.aeeab65fc7671696263825aac469a40d.css" type="text/css">
<link rel="stylesheet" href="/etc.clientlibs/core/wcm/components/search/v1/search/clientlibs/site.bc5eb8ab7795d16a0a599fbaef2adfe6.css" type="text/css">
<link rel="stylesheet" href="/etc.clientlibs/weretail/clientlibs/clientlib-base.8f235fd0c38c1c7f77b9d3b691838e68.css" type="text/css">
<link rel="stylesheet" href="/libs/foundation/clientlibs/jquery-ui/themes/default.d41d8cd98f00b204e9800998ecf8427e.css" type="text/css">

Steps to Reproduce

Links

kwin commented 4 years ago

@tlocwin Isn't this the same as #2330?

tlocwin commented 4 years ago

@kwin I was testing changes related #2330 from master and issue is still present.

kwin commented 4 years ago

Probably a different attribute needs to be used to disable the rewriting in https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/a9d8ead2840d7068bdd32f4bf0ddb920ebedf55e/bundle/src/main/java/com/adobe/acs/commons/rewriter/impl/VersionedClientlibsTransformerFactory.java#L349.

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.