Closed mogmarsh closed 2 months ago
@mogmarsh does this close #26 ?
I'd recommend also disabling remote images when not on production. Otherwise, Apple News will trigger an error and fail the push if it tries to fetch remote images that it doesn't have access to, i.e. from local envs or protected non-production envs.
@mogmarsh does this close #26 ?
it should
I'd recommend also disabling remote images when not on production. Otherwise, Apple News will trigger an error and fail the push if it tries to fetch remote images that it doesn't have access to, i.e. from local envs or protected non-production envs.
this is disabling all pushes, so the image references won't matter.
@mogmarsh Ah, right. I was thinking of scenarios in which we have test credentials on non-prod.
In that case, I'd suggest also disabling if WP_CLI
is true. That will ensure that posts don't get pushed on batch updates via CLI in production
.
@emilyatmobtown's input is valid, but should come in the form of a future issue and not block this one. I see preventing prod bulk CLI updates as a separate feature from blocking pushes on lower environments.
@emilyatmobtown's input is valid, but should come in the form of a future issue and not block this one. I see preventing prod bulk CLI updates as a separate feature from blocking pushes on lower environments.
https://github.com/alleyinteractive/wp-bulk-task/blob/main/src/trait-bulk-task-side-effects.php also could handle this in many CLI situations if we use it.
Summary
As titled.
Notes for reviewers
None.
Changelog entries
Added
Changed
Deprecated
Removed
Fixed
Security