Skip to content

Commit 3d34960

Browse files
Coponsmatticbot
authored andcommitted
AI Launchpad: sidebar-top menu, entity decoding, id_map task dedupe (#50704)
* AI Launchpad: move the Site Setup menu item to the top of the sidebar Site Setup is the primary surface while onboarding, so it now sits above Dashboard (menu position 1) instead of just below it. * AI Launchpad: decode HTML entities in user-generated content blogname/blogdescription are stored HTML-escaped (sanitize_option runs esc_html on save), and React escapes again on render, so titles like "Copons' Travelogue" showed literal entities in the wizard Name and description prefills and the preview-card title. Decode with @wordpress/html-entities at the two places API data enters the client: the app's initial read and the tailored list's refetch. This also stops progressive double-escaping: finishing the wizard with an unedited encoded prefill used to write the entities back through the PUT, re-escaping them on every pass. * AI Launchpad: collapse id_map twin tasks onto a single id Four catalog id_map pairs sat on the AI task menu as two ids for the same underlying task, so the model could offer both and users could see near-duplicate cards. The menu now offers one of each pair — the one with the stronger title/behavior: - connect_social_media over drive_traffic (concrete title, already the post_sharing_enabled remap target and CTA-override entry) - first_post_published over first_post_published_newsletter (draft-aware title; the AI subtitle carries the newsletter flavor) - site_launched over link_in_bio_launched (identical titles, canonical id) - import_subscribers over subscribers_added (the latter is gated on a Calypso onboarding goal AI Launchpad users don't have) Dropped ids stay valid on PUT and remap onto the kept twin on read (the never-offered videopress_launched twin included), so persisted outputs and stray AI emissions keep rendering; skips, listeners, analytics, and the tracked_completed reporting baseline all follow the remap, keeping completion events from re-firing under the new id. drive_traffic's backfill-pool slot goes to mobile_app_installed, whose completion is computed and self-caching, keeping the pool at the four fillers the six-task floor needs; pool candidates now build one at a time and stop at the target so its remote completion lookup is only paid when the cheaper fillers were not enough. Committed via a GitHub action: https://github.com/Automattic/jetpack/actions/runs/29910617692 Upstream-Ref: Automattic/jetpack@bcb3d9e
1 parent 8363ce7 commit 3d34960

2 files changed

Lines changed: 8 additions & 8 deletions

File tree

dist/index.d.cts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -946,7 +946,7 @@ declare const BarListChartResponsive: ({
946946
width,
947947
height,
948948
...chartProps
949-
}: Omit<BarListChartProps, "width" | "height" | "size"> & {
949+
}: Omit<BarListChartProps, "size" | "width" | "height"> & {
950950
width?: number;
951951
height?: number;
952952
size?: number;
@@ -1100,7 +1100,7 @@ declare const GeoChartResponsive: ({
11001100
width,
11011101
height,
11021102
...chartProps
1103-
}: Omit<GeoChartProps, "width" | "height" | "size"> & {
1103+
}: Omit<GeoChartProps, "size" | "width" | "height"> & {
11041104
width?: number;
11051105
height?: number;
11061106
size?: number;
@@ -1271,7 +1271,7 @@ declare const LeaderboardChartResponsive: (({
12711271
width,
12721272
height,
12731273
...chartProps
1274-
}: Omit<LeaderboardChartProps, "width" | "height" | "size"> & {
1274+
}: Omit<LeaderboardChartProps, "size" | "width" | "height"> & {
12751275
width?: number;
12761276
height?: number;
12771277
size?: number;
@@ -1804,7 +1804,7 @@ declare const Sparkline: ({
18041804
width,
18051805
height,
18061806
...chartProps
1807-
}: Omit<SparklineProps, "width" | "height" | "size"> & {
1807+
}: Omit<SparklineProps, "size" | "width" | "height"> & {
18081808
width?: number;
18091809
height?: number;
18101810
size?: number;

dist/index.d.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -946,7 +946,7 @@ declare const BarListChartResponsive: ({
946946
width,
947947
height,
948948
...chartProps
949-
}: Omit<BarListChartProps, "width" | "height" | "size"> & {
949+
}: Omit<BarListChartProps, "size" | "width" | "height"> & {
950950
width?: number;
951951
height?: number;
952952
size?: number;
@@ -1100,7 +1100,7 @@ declare const GeoChartResponsive: ({
11001100
width,
11011101
height,
11021102
...chartProps
1103-
}: Omit<GeoChartProps, "width" | "height" | "size"> & {
1103+
}: Omit<GeoChartProps, "size" | "width" | "height"> & {
11041104
width?: number;
11051105
height?: number;
11061106
size?: number;
@@ -1271,7 +1271,7 @@ declare const LeaderboardChartResponsive: (({
12711271
width,
12721272
height,
12731273
...chartProps
1274-
}: Omit<LeaderboardChartProps, "width" | "height" | "size"> & {
1274+
}: Omit<LeaderboardChartProps, "size" | "width" | "height"> & {
12751275
width?: number;
12761276
height?: number;
12771277
size?: number;
@@ -1804,7 +1804,7 @@ declare const Sparkline: ({
18041804
width,
18051805
height,
18061806
...chartProps
1807-
}: Omit<SparklineProps, "width" | "height" | "size"> & {
1807+
}: Omit<SparklineProps, "size" | "width" | "height"> & {
18081808
width?: number;
18091809
height?: number;
18101810
size?: number;

0 commit comments

Comments
 (0)