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

Fix space in custom migration #592

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gianniftp
Copy link

on plone 5.0.10 (python 2.7.18) i'm migrating some AT contents.
Using the @@custom_migration form there is a problem with spaces in at names.
the problem is already known in an issue solved in plone.app.contenttype 1.1.x, but non fixed in next releases (#305).

the fix proposed isn't enough, as also the javascript function that manage the fields listing in the form is faulty in managing spaces in types names.

this could fix the @@custom_migration form

gianni added 2 commits March 10, 2021 12:45
…ethod, useful when multiple spaces are in type name
…m request;

added changelog in news to refer to the issue plone#305.
@mister-roboto
Copy link

gianni your emails are not known to GitHub and thus it is impossible to know if you have signed the Plone Contributor Agreement, which is required to merge this pull request.

Learn about the Plone Contributor Agreement: http://docs.plone.org/develop/coredev/docs/contributors_agreement_explained.html How to add more emails to your GitHub account: https://help.github.com/articles/adding-an-email-address-to-your-github-account/

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