Automattic / Automattic/WP-Job-Manager
Make Jetpack Publicize opt-in rather than opt-out
- Dominant language
- PHP
- Stars
- 899
- Forks
- 369
- Avg merge
- 11h 37m
- Merged PRs (30d)
- 12
Description
WPJM's Job Listing CPT opts into Jetpack's Publicize by default:
https://github.com/Automattic/WP-Job-Manager/blob/master/includes/class-wp-job-manager-post-types.php#L284
The problem is that this automatically posts new Job Listings to Publicize for sites that likely intended to use Publicize for blog posts. When posting a Job Listing from WP Admin, the user may choose not to publicize it, but all posts created from the frontend are publicized by default.
The site owner can opt out of this by using a filter, but I think that this feature should probably be opt-in, given the automatic and all-encompassing nature of Publicize.
cc @jom
Contributor guide
Research direction
Start in includes/class-wp-job-manager-post-types.php around line 284, where the Job Listing CPT opts into Jetpack Publicize. Trace the existing post-type setting and its filter behavior. Done means new Job Listings, including those created from the frontend, are not publicized by default while site owners can explicitly enable the behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100