lukkien / wagtail-marketing-addons Goto Github PK

• Getting started
• Available settings
• Score rationale
• More about the what and why

In an occasion where an Excel list is handed over there is a possibility where a duplicate from-record exists. For example, having a file with over 1.000 records could have the following:

When this is present the importer will throw an error as it tries to save another record in which old_path is classified unique:

duplicate key value violates unique constraint "wagtailredirects_redirect_old_path_site_id_783622d7_uniq"
DETAIL:  Key (old_path, site_id)=(/my/previous/location/1, 3) already exists.
mechanism

Proposed solution
We should wrap this in a try-catch, process the file accordingly and notify we have skipped a record due to a duplication.

Currently there is no way to connect a list of redirects to a specific or all sites. This in inconvenient when having multiple sites available within in the same platform.

Might be a matter of rendering the breadcrumb somewhere

Currently the listing view is serving all pages, it should respect PagePermission

Right now when initialising the marketing add-ons we validate if either the modeladmin and redirects is present within the current running application. Yet these are still features you can opt-in to when needed within Wagtail. Maybe the SEO listing is not relevant when you only want to use the bulk import but it will show-up in the navigation and it will require you to add the modeladmin to your INSTALLED_APPS.

Proposed solution

• Place the modules in dedicated folders
• Validate per installed module if the requirements are present
• Take sub-dependency of a module into consideration

As of now, scores are displayed on a range of four emojis: 😏 😱 😢 😄

Without a legend or a title it is quite hard to see if the score is good or bad.

Possible solutions could be a change to a star-based (e.g. ⭐️-⭐️⭐️⭐️⭐️) or numerical (e.g. 0/10 - 10/10) rating

• Test edit-url resolving
• Test score resolving
• Test custom page model resolving
• Quick test custom properties on view