-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Open
Labels
break-me-upbreak up for cleaner code separation, discrete tests, and, easier and iterative collaborationbreak up for cleaner code separation, discrete tests, and, easier and iterative collaborationdeprecationenhancementexplains that the issue is to improve upon one of our existing featuresexplains that the issue is to improve upon one of our existing featureshelp wantedrequires help by anyone willing to contributerequires help by anyone willing to contribute
Milestone
Description
Some nodes use a legacy main image type: DrupalNodeMainImage -- not sure how many are left.
Example: http://publiclab.org/wiki/somerville-massachusetts (though this may have changed?)
This could be done by:
- writing a query which returns all nodes with a DrupalMainImage (by using logic like this: https://github.com/publiclab/plots2/blob/master/app/models/drupal_node.rb#L296-L308)
- by using this code in a Rails Migration to create a new native Image from the path of the node's
DrupalNodeMainImage: https://github.com/publiclab/plots2/blob/master/app/controllers/images_controller.rb#L10-L13 - deleting all legacy DrupalMainImage code, mostly in this method and elsewhere in that file: https://github.com/publiclab/plots2/blob/master/app/models/drupal_node.rb#L296-L308
These steps could be fleshed out and broken up more into separate issues.
Metadata
Metadata
Assignees
Labels
break-me-upbreak up for cleaner code separation, discrete tests, and, easier and iterative collaborationbreak up for cleaner code separation, discrete tests, and, easier and iterative collaborationdeprecationenhancementexplains that the issue is to improve upon one of our existing featuresexplains that the issue is to improve upon one of our existing featureshelp wantedrequires help by anyone willing to contributerequires help by anyone willing to contribute