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

Social: Clean up media auto-conversion backend logic #38587

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
f52d06f
Social: Clean up media auto-conversion backend logic
manzoorwanijk Jul 29, 2024
2f64072
Add changelog
manzoorwanijk Jul 29, 2024
8a1046a
Fix uncached call via delete_option
manzoorwanijk Jul 30, 2024
358aa84
Fix unit tests
manzoorwanijk Jul 30, 2024
6e18ccc
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Jul 30, 2024
575be91
Fix up versions
manzoorwanijk Jul 30, 2024
c0d02c2
Update class-publicize-base.php
manzoorwanijk Jul 30, 2024
06fbd32
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Jul 30, 2024
65148bf
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Jul 30, 2024
f6e28f0
Do what CI should be doing - fix up versions
manzoorwanijk Jul 30, 2024
2e87d75
Add unnecessary changelogs
manzoorwanijk Jul 30, 2024
3d01d73
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Jul 31, 2024
ad18e93
Remove unnecessary changelog
manzoorwanijk Jul 31, 2024
ec1da6f
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Aug 6, 2024
03cd449
Fix up versions
manzoorwanijk Aug 6, 2024
e481e1b
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Aug 6, 2024
67d29c4
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Aug 7, 2024
a4fe44d
Fix up versions
manzoorwanijk Aug 7, 2024
8a2f909
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Aug 9, 2024
bb8bc67
Fix up versions
manzoorwanijk Aug 9, 2024
5881f2e
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Aug 12, 2024
17e2347
Fix up versions
manzoorwanijk Aug 12, 2024
b836196
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Aug 14, 2024
4470b86
Fix up versions
manzoorwanijk Aug 14, 2024
b2f820f
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Aug 19, 2024
b1ec968
Fix up versions
manzoorwanijk Aug 19, 2024
c26899d
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Aug 20, 2024
7c17c8f
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Aug 21, 2024
a1951d0
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Aug 21, 2024
a77fe85
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Aug 27, 2024
81d1b19
Fix up versions
manzoorwanijk Aug 27, 2024
09b8ed5
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Aug 28, 2024
d69fb8f
Fix up versions
manzoorwanijk Aug 28, 2024
daa8d54
Remove unnecessary changelogs
manzoorwanijk Aug 29, 2024
b5046e4
Delete unnecessary changelog
manzoorwanijk Aug 29, 2024
fc35fde
Merge branch 'trunk' into update/social/clean-up-media-auto-conversio…
manzoorwanijk Sep 5, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions projects/packages/publicize/.phan/baseline.php
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@

// Currently, file_suppressions and directory_suppressions are the only supported suppressions
'file_suppressions' => [
'src/auto-conversion-settings/class-rest-settings-controller.php' => ['PhanPluginMixedKeyNoKey'],
'src/class-connections-post-field.php' => ['PhanPluginDuplicateConditionalNullCoalescing'],
'src/class-keyring-helper.php' => ['PhanTypeMismatchArgumentProbablyReal', 'PhanTypeMismatchDefault'],
'src/class-publicize-base.php' => ['PhanImpossibleCondition', 'PhanPluginDuplicateConditionalNullCoalescing', 'PhanPluginSimplifyExpressionBool', 'PhanSuspiciousMagicConstant', 'PhanTypeMismatchArgument', 'PhanTypeMismatchArgumentNullable', 'PhanTypeMismatchArgumentNullableInternal', 'PhanTypeMismatchDimFetch', 'PhanTypeMismatchReturn'],
Expand All @@ -42,7 +41,6 @@
'src/social-image-generator/class-rest-settings-controller.php' => ['PhanPluginMixedKeyNoKey'],
'src/social-image-generator/class-settings.php' => ['PhanPluginDuplicateConditionalNullCoalescing'],
'src/social-image-generator/class-setup.php' => ['PhanTypeMismatchArgumentNullable'],
'tests/php/jetpack-social-settings/test-auto-conversion.php' => ['PhanDeprecatedFunction', 'PhanPluginDuplicateConditionalNullCoalescing'],
'tests/php/jetpack-social-settings/test-jetpack-social-settings.php' => ['PhanDeprecatedFunction'],
'tests/php/test-connections-post-field.php' => ['PhanDeprecatedFunction', 'PhanTypeMismatchArgument'],
'tests/php/test-publicize-og-optimization.php' => ['PhanDeprecatedFunction'],
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
Significance: minor
Type: removed

Social: Cleaned up media auto-conversion backend logic

This file was deleted.

This file was deleted.

2 changes: 2 additions & 0 deletions projects/packages/publicize/src/class-publicize-base.php
Original file line number Diff line number Diff line change
Expand Up @@ -2028,6 +2028,8 @@ public function has_social_image_generator_feature() {
/**
* Check if the auto-conversion feature is one of the active features.
*
* TODO: Remove this after certain releases of Jetpack v15.
*
* @param string $type Whether image or video.
*
* @return bool
Expand Down
4 changes: 0 additions & 4 deletions projects/packages/publicize/src/class-publicize-setup.php
Original file line number Diff line number Diff line change
Expand Up @@ -46,10 +46,6 @@ public static function on_jetpack_feature_publicize_enabled() {
add_action( 'rest_api_init', array( static::class, 'register_core_options' ) );
add_action( 'admin_init', array( static::class, 'register_core_options' ) );

// Flagged to be removed after deprecation.
// @deprecated 0.38.3
add_action( 'rest_api_init', array( new Auto_Conversion\REST_Settings_Controller(), 'register_routes' ) );

( new Social_Image_Generator\Setup() )->init();
}

Expand Down
2 changes: 1 addition & 1 deletion projects/packages/publicize/src/class-rest-controller.php
Original file line number Diff line number Diff line change
Expand Up @@ -339,7 +339,7 @@ public function get_dismiss_notice_endpoint_schema() {
'notice' => array(
'description' => __( 'Name of the notice to dismiss', 'jetpack-publicize-pkg' ),
'type' => 'string',
'enum' => array( 'instagram', 'advanced-upgrade-nudge-admin', 'advanced-upgrade-nudge-editor', 'auto-conversion-editor-notice' ),
'enum' => array( 'instagram', 'advanced-upgrade-nudge-admin', 'advanced-upgrade-nudge-editor' ),
'required' => true,
),
'reappearance_time' => array(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ public function register() {
'schema' => array(
'type' => 'object',
'properties' => array(
'auto-conversion-editor-notice' => array( 'type' => 'number' ),
'advanced-upgrade-nudge-admin' => array( 'type' => 'number' ),
'advanced-upgrade-nudge-editor' => array( 'type' => 'number' ),
),
Expand Down
Loading
Loading