Code review comment for lp://staging/~doctormo/inkscape-web/inkscape-web-inkpicture

Revision history for this message
Hachmann (marenhachmann) wrote :

Beside the missing closing ) in the comment on the plugin, it looks almost perfect.

The 'longdesc' field has not been removed (as the migration thinks), it has been renamed to what it is: title.

This does loose data on the website, as I added titles (at least) on the Sponsors page (how did you check for data loss?). So I guess it will need to be named back to its original name, and only the 'verbose' and 'help text' parts need to be changed.

Why is compatibility with the old plugin important? I liked the name ;-)

« Back to merge proposal