Skip to content

Remove Legacy Landing Pages from Settings Dropdowns - #1153

Merged
n7studios merged 7 commits into
remove-legacy-forms-dropdownfrom
remove-legacy-landing-pages-dropdown
Aug 5, 2026
Merged

Remove Legacy Landing Pages from Settings Dropdowns#1153
n7studios merged 7 commits into
remove-legacy-forms-dropdownfrom
remove-legacy-landing-pages-dropdown

Conversation

@n7studios

Copy link
Copy Markdown
Contributor

Summary

Removes legacy Kit landing pages from every settings dropdown across the plugin, while preserving any legacy landing page (by ID or by pre-1.9.6 URL string) already saved to a page's post meta.

Testing

Tests added:

Integration (tests/Integration/ResourceLandingPagesTest.php):

  • testGetNonLegacyget_non_legacy() returns v4 landing pages and excludes legacy; every returned page has embed_url and no url key
  • testNonLegacyExist — helper returns true when v4 pages exist
  • testIsLegacyWithLegacyID — true for legacy ID
  • testIsLegacyWithURLString — true when passed a URL string (pre-1.9.6 storage)
  • testIsLegacyWithV4ID — false for v4 ID
  • testIsLegacyWithInvalidID — false for an ID that doesn't resolve

End-to-end:

  • ClassicEditorLandingPageCest::testLegacyLandingPagesExcludedFromMetaboxDropdown — Classic editor metabox Landing Page dropdown has no legacy option
  • ClassicEditorLandingPageCest::testMetaboxPreservesSelectedLegacyLandingPageByID — page preseeded with legacy ID in post meta shows as selected in metabox AND still renders on the frontend, including the WordPress site icon replacing the Kit favicon
  • ClassicEditorLandingPageCest::testMetaboxPreservesSelectedLegacyLandingPageByURL — page preseeded with a legacy URL string (pre-1.9.6 storage) shows as selected in metabox
  • BlockEditorLandingPageCest::testLegacyLandingPagesExcludedFromPluginSidebar — Gutenberg plugin sidebar Landing Page dropdown has no legacy option
  • BlockEditorLandingPageCest::testPluginSidebarPreservesSelectedLegacyLandingPage — page preseeded with legacy ID shows as selected in Gutenberg plugin sidebar
  • PageLandingPageSetupWizardCest::testLegacyLandingPagesExcludedFromSetupWizard — Setup Wizard landing page picker has no legacy option

Tests removed (all previously selected a legacy landing page from a dropdown that no longer offers it):

  • ClassicEditorLandingPageCest::testAddNewPageUsingDefinedLegacyLandingPage
  • ClassicEditorLandingPageCest::testLegacyLandingPageSiteIcon (site-icon coverage moved into testMetaboxPreservesSelectedLegacyLandingPageByID)
  • BlockEditorLandingPageCest::testAddNewPageUsingDefinedLegacyLandingPage
  • BlockEditorLandingPageCest::testLegacyLandingPageSiteIcon
  • PageLandingPageSetupWizardCest::testAddNewLegacyLandingPage

Checklist

@n7studios n7studios added this to the 3.3.7 milestone Jul 29, 2026
@n7studios n7studios self-assigned this Jul 29, 2026
@n7studios n7studios modified the milestones: 3.3.7, 3.3.8 Jul 31, 2026
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@n7studios
n7studios requested review from a team, ciccio-kit and noelherrick and removed request for a team August 4, 2026 13:24
@n7studios
n7studios marked this pull request as ready for review August 4, 2026 13:24

@ciccio-kit ciccio-kit left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ✅

Comment thread views/backend/post/meta-box.php Outdated
Comment thread includes/class-convertkit-resource-landing-pages.php
@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@n7studios
n7studios merged commit 4994137 into remove-legacy-forms-dropdown Aug 5, 2026
312 of 344 checks passed
@n7studios
n7studios deleted the remove-legacy-landing-pages-dropdown branch August 7, 2026 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants