Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove outdated features #179

Closed
enricobattocchi opened this issue Feb 3, 2023 · 3 comments · Fixed by #180
Closed

Remove outdated features #179

enricobattocchi opened this issue Feb 3, 2023 · 3 comments · Fixed by #180
Assignees

Comments

@enricobattocchi
Copy link
Member

Describe the feature you'd like

We should revert or hide:

@vraja-pro vraja-pro self-assigned this Feb 6, 2023
@enricobattocchi
Copy link
Member Author

2 things:

  1. See https://yoast.slack.com/archives/C03KU0EHCNQ/p1675771147990779?thread_ts=1675694734.622359&cid=C03KU0EHCNQ so we want to keep the threshold part, but hidden
  2. that section wasn't actually removed in the PR, but made visible since the check on the flag was removed.

I'd reinstate the check on the flag then

@enricobattocchi
Copy link
Member Author

So we need to reinstate the indexables page feature toggle and only remove the settings UI toggle instead for now.
We'll revise this when the indexables page is removed from Free.

@enricobattocchi
Copy link
Member Author

We decided to remove the Indexables Page from Free, so we're back to the removal in the Test Helper as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants