search

coderedcorp / wagtail-seo

Search engine and social media optimization for Wagtail.
Other
66 stars 18 forks source link

feat: split struct org data from SeoMixin (mixin) #64

Closed Chadys closed 1 month ago

Chadys commented 4 months ago

Fix #60 An alternative solution from #63 to remove struct data from SeoMixin, by creating SeoOrgMixin instead.

I find this solution much less risky, the impact for the upgrade is lessened compared to the settings approach, there was no need to change tests or templates and no manual code is required for the migrations.

vsalvino commented 4 months ago

Amazing, thank you. I will review these two hopefully at the Wagtail Space US sprints :)

vsalvino commented 1 month ago

Merging this in via #71