search

torchbox / wagtail-wordpress-import

A package for Wagtail CMS to import WordPress blog content from an XML file into Wagtail
MIT License
44 stars 16 forks source link

I think these module paths are a bit long #39

Closed nickmoreton closed 3 years ago

nickmoreton commented 3 years ago

I think these module paths are a bit long, would it be possible to re-export them from wagtail_wordpress_import.prefilters to make them shorter and more memorable? For example:

WAGTAIL_WORDPRESS_IMPORT_PREFILTERS = [
    {
        "FUNCTION": "wagtail_wordpress_import.prefilters.linebreaks_wp",
    },
    {
        "FUNCTION": "wagtail_wordpress_import.prefilters.normalize_style_attrs",
    },
    {
        "FUNCTION": "wagtail_wordpress_import.prefilters.fix_styles",
    },
    {
        "FUNCTION": "wagtail_wordpress_import.prefilters.bleach_clean",
    },
]

Also, think we could pick a better name for fix_styles. How about styles_to_tags?

_Originally posted by @kaedroho in https://github.com/torchbox/wagtail-wordpress-import/pull/36#discussion_r726486501_

nickmoreton commented 3 years ago

Look at renaming / shortening imports

nickmoreton commented 3 years ago

Fixed in https://github.com/torchbox/wagtail-wordpress-import/pull/35