@wix/himalaya-cli 0.1253.0 → 0.1254.0

package/templates/yoga-studio-owner/tier5-src/himi-owner-lite-home/index.ts

@@ -10,6 +10,7 @@ import { countSites } from "@himalaya/wix-site-list";
 import { searchOrders, type WixOrder } from "@himalaya/wix-ecom";
 import { queryBookings } from "@himalaya/wix-bookings";
 import { UPCOMING_BOOKING_STATUSES, upcomingBookingCount } from "../_shared/owner-bookings.js";
+import { ownerOrderAttentionFilter } from "../_shared/owner-order-attention.js";
 import { queryMembers } from "@himalaya/wix-members";
 import { queryReviews } from "@himalaya/wix-reviews";
 import { queryReservations } from "@himalaya/wix-table-reservations";
@@ -102,8 +103,6 @@ const UPCOMING_RESERVATION_STATUSES = ["HELD", "RESERVED", "REQUESTED"];
 const UPCOMING_EVENT_STATUSES = ["UPCOMING", "SCHEDULED"];
 const ATTENTION_MEMBER_STATUSES = ["PENDING"];
 const ATTENTION_REVIEW_STATUSES = ["IN_MODERATION", "SUBMITTED"];
-const ATTENTION_PAYMENT_STATUSES = ["NOT_PAID", "PARTIALLY_PAID"];
-const ATTENTION_FULFILLMENT_STATUSES = ["NOT_FULFILLED", "PARTIALLY_FULFILLED"];
 const SOURCE_LABELS: Record<HomeSourceId, string> = {
   products: "Products", posts: "Posts", events: "Events", sites: "Sites",
   orders: "Orders", bookings: "Bookings", members: "Members",
@@ -469,14 +468,7 @@ async function loadSourcePart(source: HomeSourceId, part: SourcePart): Promise<S
     case "sites": return countSites({}) as Promise<SourcePayload>;
     case "orders": {
       if (part === "primary") return searchOrders({ search: {
-          filter: {
-            archived: false,
-            status: { $nin: ["CANCELED", "CANCELLED"] },
-            $or: [
-              { paymentStatus: { $in: ATTENTION_PAYMENT_STATUSES } },
-              { paymentStatus: "PAID", fulfillmentStatus: { $in: ATTENTION_FULFILLMENT_STATUSES } },
-            ],
-          },
+          filter: ownerOrderAttentionFilter(),
           sort: [{ fieldName: "createdDate", order: "DESC" }, { fieldName: "id", order: "ASC" }],
           cursorPaging: { limit: TODAY_PAGE },
         } }) as Promise<SourcePayload>;

package/templates/yoga-studio-owner/tier5-src/himi-owner-lite-search/index.ts

@@ -213,7 +213,10 @@ function copyFromParams(): SearchCopy {
     allFailed: copyParam("ownerSearch.allFailed", DEFAULT_COPY.allFailed),
     loadedOnly: copyParam("ownerSearch.loadedOnly", DEFAULT_COPY.loadedOnly),
     loadingSummary: copyParam("ownerSearch.loading", DEFAULT_COPY.loadingSummary),
-    emptySummary: copyParam("ownerSearch.empty", DEFAULT_COPY.emptySummary),
+    // Keep the pre-worker-heading copy key alive for existing copy packs. The
+    // newer `empty` key takes precedence when present; both flow through the
+    // query interpolator so neither can hide what the owner searched.
+    emptySummary: copyParam("ownerSearch.empty", copyParam("ownerSearch.emptyTitle", DEFAULT_COPY.emptySummary)),
     idleSummary: copyParam("ownerSearch.idle", DEFAULT_COPY.idleSummary),
     permission: copyParam("ownerSearch.permission", DEFAULT_COPY.permission),
     setup: copyParam("ownerSearch.setup", DEFAULT_COPY.setup),
@@ -240,6 +243,7 @@ const IDLE: State = {
   anyLoading: false,
   showEmpty: false,
   hideEmpty: true,
+  emptyTitle: "",
 };
 
 /** Rebuilt on `load`, because the allowlist arrives with the descriptor. */

package/templates/yoga-studio-owner/tier5-src/himi-owner-lite-search/logic.ts

@@ -106,6 +106,14 @@ export interface SearchItem {
   /** Stable across refreshes, so a re-fetch cannot re-key a row under the
    *  user's finger: `h:<source>`, `r:<source>:<id>`, `f:<source>`. */
   key: string;
+  /** The row's LIST identity — always equal to `key`, never blank.
+   *
+   *  A bound list keys its rows on `id` (the web BoundList does, verbatim). This
+   *  carried the record id, which left every heading and footer at `""` and let
+   *  two domains that reuse a record id collide — so React reconciled headings
+   *  onto each other as groups landed and painted "Contacts" three times.
+   *  What a record OPENS is `recordId`. */
+  id: string;
   /** The source this row belongs to — a retry or a pager addresses it. */
   sourceId: string;
   // Exactly one kind is true. Pre-negated, because a `hiddenWhen` takes one path.
@@ -122,7 +130,9 @@ export interface SearchItem {
   isLoading: boolean;
   notLoading: boolean;
   // Record.
-  id: string;
+  /** The record this row opens — the destination loads it by this id. Blank on
+   *  headings and footers, which open nothing. */
+  recordId: string;
   title: string;
   subtitle: string;
   statusLabel: string;
@@ -163,6 +173,10 @@ export interface SearchState {
    *  nobody has typed into yet, and again over every result still in flight. */
   showEmpty: boolean;
   hideEmpty: boolean;
+  /** The empty state's heading, naming the query ("No records match “mug”").
+   *  Blank unless `showEmpty` — and while it shows, `summary` is blank, because
+   *  the two sat one above the other saying the same sentence. */
+  emptyTitle: string;
 }
 
 export interface SearchCopy {
@@ -192,7 +206,9 @@ export const DEFAULT_COPY: SearchCopy = {
   allFailed: "No area could be searched",
   loadedOnly: "{n} in loaded records",
   loadingSummary: "Searching…",
-  emptySummary: "No records matched",
+  /** The empty state's heading. `{q}` is the query the owner typed, so the
+   *  state says WHAT matched nothing rather than only that something did not. */
+  emptySummary: "No records match “{q}”",
   idleSummary: "Search your business records",
   permission: "You don’t have access to this area",
   setup: "This area isn’t set up on this site",
@@ -231,6 +247,14 @@ function fill(template: string, n: number, token = "{n}"): string {
   return template.includes(token) ? template.split(token).join(String(n)) : `${template} ${n}`;
 }
 
+/** `{q}` → the query. Custom copy stays query-specific even when it omits the
+ *  placeholder, so the empty state always identifies what was searched. */
+function fillQuery(template: string, query: string): string {
+  return template.includes("{q}")
+    ? template.split("{q}").join(query)
+    : `${template}: “${query}”`;
+}
+
 function idleGroup(adapter: SearchAdapter): SearchGroup {
   return decorate({
     id: adapter.id,
@@ -273,9 +297,9 @@ function baseItem(sourceId: string): SearchItem {
     isHeader: false, notHeader: true,
     isRecord: false, notRecord: true,
     isFooter: false, notFooter: true,
-    label: "", countLabel: "",
+    id: "", label: "", countLabel: "",
     isLoading: false, notLoading: true,
-    id: "", title: "", subtitle: "", statusLabel: "", image: "", screenId: "",
+    recordId: "", title: "", subtitle: "", statusLabel: "", image: "", screenId: "",
     error: "", hasError: false, noError: true,
     canLoadMore: false, cannotLoadMore: true, isLoadingMore: false,
   };
@@ -302,7 +326,7 @@ export function projectItems(groups: SearchGroup[]): SearchItem[] {
     if (silent) continue;
     items.push({
       ...baseItem(group.id),
-      key: `h:${group.id}`,
+      key: `h:${group.id}`, id: `h:${group.id}`,
       isHeader: true, notHeader: false,
       label: group.label,
       // A count is only shown once there is something to count: "0" under a
@@ -313,9 +337,9 @@ export function projectItems(groups: SearchGroup[]): SearchItem[] {
     for (const row of group.rows) {
       items.push({
         ...baseItem(group.id),
-        key: `r:${group.id}:${row.id}`,
+        key: `r:${group.id}:${row.id}`, id: `r:${group.id}:${row.id}`,
         isRecord: true, notRecord: false,
-        id: row.id, title: row.title, subtitle: row.subtitle,
+        recordId: row.id, title: row.title, subtitle: row.subtitle,
         statusLabel: row.statusLabel, image: row.image, screenId: row.screenId,
       });
     }
@@ -325,7 +349,7 @@ export function projectItems(groups: SearchGroup[]): SearchItem[] {
     if (group.hasError || group.problem !== "" || group.canLoadMore) {
       items.push({
         ...baseItem(group.id),
-        key: `f:${group.id}`,
+        key: `f:${group.id}`, id: `f:${group.id}`,
         isFooter: true, notFooter: false,
         error: group.error,
         hasError: group.error !== "", noError: group.error === "",
@@ -393,7 +417,9 @@ export function createGroupedSearch(
       : anyLoading && total === 0
         ? copy.loadingSummary
         : total === 0
-          ? (anyAnswered ? copy.emptySummary : copy.allFailed)
+          // Empty and answered: the empty block carries the sentence, so the
+          // summary line above it steps aside rather than repeating it.
+          ? (anyAnswered ? "" : copy.allFailed)
           : total === 1
             ? copy.resultsOne
             : areas === 1
@@ -416,6 +442,7 @@ export function createGroupedSearch(
       anyLoading,
       showEmpty: emptyAndAnswered,
       hideEmpty: !emptyAndAnswered,
+      emptyTitle: emptyAndAnswered ? fillQuery(copy.emptySummary, query) : "",
     };
   };