Skip to content

Remove the now unused onboarding code#1124

Merged
dkotter merged 5 commits into
developfrom
remove/onboarding
Jun 24, 2026
Merged

Remove the now unused onboarding code#1124
dkotter merged 5 commits into
developfrom
remove/onboarding

Conversation

@dkotter

@dkotter dkotter commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

Description of the Change

In #411 we added a new user onboarding experience that would guide a user through setup after they first activated the plugin.

In #771 we refactored our settings page significantly and as part of that, we removed the onboarding from ever loading in that new experience. The only ones that could access that onboarding were those that had deliberately used the classifai_use_legacy_settings_panel filter, which is now over 2 years old and will be removed soon.

This means the onboarding code has lived within ClassifAI for the last 2 years but hasn't actually been used by most users (if by any). As such, this PR removes all of that code.

How to test the Change

Not much to test here other than to ensure the normal settings experience still works as expected

Changelog Entry

Removed - New user onboarding experience that has only been accessible to those that are using the classifai_use_legacy_settings_panel filter set to true.

Credits

Props @dkotter

Checklist:

Open WordPress Playground Preview

@dkotter dkotter added this to the 3.9.0 milestone Jun 23, 2026
@dkotter dkotter self-assigned this Jun 23, 2026
@dkotter dkotter requested review from a team and jeffpaul as code owners June 23, 2026 20:03
@github-actions github-actions Bot added the needs:code-review This requires code review. label Jun 23, 2026
@github-actions

Copy link
Copy Markdown

✅ WordPress Plugin Check Report

✅ Status: Passed

📊 Report

All checks passed! No errors or warnings found.


🤖 Generated by WordPress Plugin Check Action • Learn more about Plugin Check

@jeffpaul jeffpaul left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Did some light testing in Playground and things worked as expected.

@dkotter dkotter merged commit ffa6945 into develop Jun 24, 2026
33 checks passed
@dkotter dkotter deleted the remove/onboarding branch June 24, 2026 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:code-review This requires code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants