Currently we are changing a lot of things related to content tracking & tasks.
In conversation with @aristath we mentioned that Content_Helpers class will probably be remove as not needed anymore.
In parallel we are adding valuable content settings and tasks in following PRs:
For now I have copied Content_Helpers::get_post_names() to class-settings.php in one of the branches so that helper is not removed if Content_Helpers is deleted, but we should find a better place for it once refactor is done (and also use it in all above PRs consistently).
Using just \progress_planner()->get_settings()->get( [ 'include_post_types' ], [ 'post', 'page'] ) gets the saved value properly, but since CPT properties (like public) can be changed dynamically I think it's better to have a helper in place which filters out CPTs which are no longer eligible.
Currently we are changing a lot of things related to content tracking & tasks.
In conversation with @aristath we mentioned that Content_Helpers class will probably be remove as not needed anymore.
In parallel we are adding
valuable contentsettings and tasks in following PRs:For now I have copied Content_Helpers::get_post_names() to class-settings.php in one of the branches so that helper is not removed if Content_Helpers is deleted, but we should find a better place for it once refactor is done (and also use it in all above PRs consistently).
Using just
\progress_planner()->get_settings()->get( [ 'include_post_types' ], [ 'post', 'page'] )gets the saved value properly, but since CPT properties (likepublic) can be changed dynamically I think it's better to have a helper in place which filters out CPTs which are no longer eligible.