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

Convert Node/Content types to CMI #171

Closed
jenlampton opened this issue Feb 7, 2014 · 4 comments · Fixed by backdrop/backdrop#507
Closed

Convert Node/Content types to CMI #171

jenlampton opened this issue Feb 7, 2014 · 4 comments · Fixed by backdrop/backdrop#507

Comments

@jenlampton
Copy link
Member

Node types need to save their settings into a config file.

For reference, here is the parallel issue from drupal.org

@jenlampton jenlampton mentioned this issue Feb 7, 2014
28 tasks
@quicksketch quicksketch changed the title Convert Nodes types to CMI Convert Node/Content types to CMI Oct 14, 2014
@quicksketch
Copy link
Member

I'm going to start working on this one, as our most important CMI conversion not yet underway.

@quicksketch
Copy link
Member

Things are coming along quickly here. We've got tests 98% passing already: https://github.com/quicksketch/backdrop/compare/171/node_config

Converting node types actually seems to be very straight-forward. They're well contained and the new approach makes managing settings significantly easier than the 20-50 variables per content type we had previously. Now all settings will be stored in a single node.type.[type_name].json file, with modules responsible for cleaning up their settings on uninstall, but not on deletion or renaming of node types. A much easier system to manage overall.

This doesn't include converting fields yet. I think we'll need to convert Taxonomy terms/vocabularies and Users to CMI first, then we can move on to fields.

@quicksketch
Copy link
Member

This PR is ready to go (IMO) at backdrop/backdrop#507.

The conversion to CMI is pretty straight-forward. I love being able to delete all the per node type variables. This PR also includes a number of updates to our JS files that deal with vertical tabs, since the form elements for a number of options have changed their names.

@quicksketch
Copy link
Member

Merged in backdrop/backdrop#507. Yay! On to Taxonomies and Roles.

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 a pull request may close this issue.

2 participants