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.

Sign up for GitHub

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

Jump to bottom

get rid of completly (in whole core) unused plone_scripts #1802

Merged
merged 2 commits into from
Oct 21, 2016
Merged

get rid of completly (in whole core) unused plone_scripts #1802

merged 2 commits into from
Oct 21, 2016

Conversation

jensens
Copy link
Member

@jensens jensens commented Oct 21, 2016

No description provided.

@jensens jensens mentioned this pull request Oct 21, 2016
Open
13 tasks
@gforcada
Copy link
Member

@jensens plone_log is still used at https://github.com/plone/Products.CMFPlone/blob/85475602e9e6d0f0861137bdea7b485ae2406e7b/Products/CMFPlone/skins/plone_form_scripts/content_status_modify.cpy

@jensens
Copy link
Member Author

jensens commented Oct 21, 2016

@gforcada I missed that one. Fun fact: it is an unused assignment. I removed it.

@gforcada gforcada closed this Oct 21, 2016
@gforcada gforcada reopened this Oct 21, 2016
@jensens jensens merged commit 6bbd14d into master Oct 21, 2016
@gforcada gforcada deleted the remove-unused-scripts branch October 22, 2016 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
04 type: enhancement 23 status: testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jensens @gforcada