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

[Plone 5.2] Fix an issue that was preventing the edit of a registry record containing a "/" in its name #53

Merged
merged 1 commit into from
Aug 10, 2021

Conversation

ale-rt
Copy link
Member

@ale-rt ale-rt commented Aug 6, 2021

Backport of #52 for Plone 5.2

Refs #51
Refs plone/Products.CMFPlone#3296

@mister-roboto
Copy link

@ale-rt thanks for creating this Pull Request and help improve Plone!

To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass.

Whenever you feel that the pull request is ready to be tested, either start all jenkins jobs pull requests by yourself, or simply add a comment in this pull request stating:

@jenkins-plone-org please run jobs

With this simple comment all the jobs will be started automatically.

Happy hacking!

@ale-rt
Copy link
Member Author

ale-rt commented Aug 6, 2021

@jenkins-plone-org please run jobs

@ale-rt ale-rt changed the title Fix an issue that was preventing the edit of a registry record contai… Fix an issue that was preventing the edit of a registry record containing a "/" in its name Aug 6, 2021
@ale-rt ale-rt changed the title Fix an issue that was preventing the edit of a registry record containing a "/" in its name [Plone 5.2] Fix an issue that was preventing the edit of a registry record containing a "/" in its name Aug 6, 2021
Copy link
Member

@mauritsvanrees mauritsvanrees left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I confirm the problem and the fix. Thanks!

@mauritsvanrees mauritsvanrees merged commit 056df78 into 1.7.x Aug 10, 2021
@mauritsvanrees mauritsvanrees deleted the 51-fix-editing-plone52 branch August 10, 2021 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants