-
-
Notifications
You must be signed in to change notification settings - Fork 188
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
Use latest z3c.form in Plone Classic UI #3459
Comments
@plone/framework-team I've created a branch with the updated z3c.form version and this what happened on jenkins ... plone/buildout.coredev#782 |
plone.autoform and plone.app.portlets can safely be merged because there are only backwards compatible test fixes. CMFPlone has a backward compatible update to login form and can be merged too. The plone(.app).z3cform package Updates need z3c.form >= 4 and therefore an updated versions.cfg of buildout.coredev … /cc @mauritsvanrees @jensens might wanna have a look? |
PS: jenkins is down to 3 failing tests. I think I can fix them later today. |
@petschki I personally don't think this requires a PLIP. Though, for what it is worth I am +1. |
+1 with this PLIP and what with Timo says about not having a PLIP for this! |
Thanks for your comments. Indeed not really worth a PLIP ... I have merged the updates in plone.(app.)z3cform packages and bumped |
PLIP (Plone Improvement Proposal)
Responsible Persons
Proposer: @petschki
Abstract
This PLIPs mission is to upgrade
z3c.form
to the latest available version in Plone Classic UI.Assumption
Right now
z3c.form
is pinned to the latest 3.x version because of the dependencies to the subform functionality provided byz3c.form.object.SubformAdapter
andz3c.form.object.ObjectSubForm
. This has been removed in z3c.form since 4.xProposal & Implementation
To get the latest z3c.form we have to backport subform functionality to
plone.z3cform
and figure out which core packages needs to be refactored.For addons we have to provide a deprecation message to the new
plone.z3cform
moduleDeliverables
Participants
The text was updated successfully, but these errors were encountered: