Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove jquery-highlightsearchterms and plone_ecmascript layer #170

Merged
merged 3 commits into from
Sep 9, 2018

Conversation

sunew
Copy link
Contributor

@sunew sunew commented Sep 6, 2018

Remove the jquery-highlightsearchterms resource, and the - now empty - plone_ecmascript skin layer. See https://github.com/plone/Products.CMFPlone/pull/1963/commits and plone/Products.CMFPlone#1811 plone/Products.CMFPlone#1801

@sunew
Copy link
Contributor Author

sunew commented Sep 6, 2018

@jensens removing the skin layer and the plone.resources/jquery-highlightsearchterms record works, but how do we remove a single value from the plone.bundles/plone-legacy record?

I tried this with no luck:

<records prefix="plone.bundles/plone-legacy" interface='Products.CMFPlone.interfaces.IBundleRegistry'> <value key="resources" purge="false"> <element remove="true">jquery-highlightsearchterms</element> </value> </records>

@sunew
Copy link
Contributor Author

sunew commented Sep 6, 2018

…- plone_ecmascript skin layer. See https://github.com/plone/Products.CMFPlone/pull/1963/commits and plone/Products.CMFPlone#1811 plone/Products.CMFPlone#1801

removing records (wip)

remove jquery-highlightsearchterms from the plone-legacy bundle

c
@sunew sunew force-pushed the remove-highlightsearchterms branch from 787704c to 6bab181 Compare September 6, 2018 16:24
@sunew sunew changed the title WIP: remove the jquery-highlightsearchterms resource, and the - now empty … Remove jquery-highlightsearchterms and plone_ecmascript layer Sep 6, 2018
@sunew sunew requested a review from cdw9 September 6, 2018 17:48
@sunew
Copy link
Contributor Author

sunew commented Sep 6, 2018

All done - for plone 5.2 and 5.1.4

@sunew sunew force-pushed the remove-highlightsearchterms branch from bac600b to c979968 Compare September 6, 2018 18:23
@sunew sunew changed the title Remove jquery-highlightsearchterms and plone_ecmascript layer [WIP] Remove jquery-highlightsearchterms and plone_ecmascript layer Sep 8, 2018
@sunew
Copy link
Contributor Author

sunew commented Sep 8, 2018

Still needs to rebuild the legacy bundle.

@sunew
Copy link
Contributor Author

sunew commented Sep 8, 2018

done

@sunew sunew changed the title [WIP] Remove jquery-highlightsearchterms and plone_ecmascript layer Remove jquery-highlightsearchterms and plone_ecmascript layer Sep 8, 2018
@sunew sunew requested a review from mauritsvanrees September 8, 2018 10:28
@jensens jensens merged commit 8d8f25d into master Sep 9, 2018
@jensens jensens deleted the remove-highlightsearchterms branch September 9, 2018 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants