fix: restore functionality from merge

[colton-demetriou]

tested here: https://www.yext.com/s/4259018/yextsites/161497/branches/131426/editor#pageSetId=empty-loc-test&locale=en&themeId=empty-test&entityId=1095527849

Fix reviews section, slotify nearby locations empty state logic, and restore migration logic that was removed

[github-actions]

Warning: Component files have been updated but no migrations have been added. See https://github.com/yext/visual-editor/blob/main/packages/visual-editor/src/components/migrations/README.md for more information.

[pkg-pr-new]

pages-visual-editor-starter

npm i https://pkg.pr.new/yext/visual-editor/@yext/visual-editor@882

commit: cdb62b9

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request introduces empty-state handling and data migration features across multiple components. The NearbyLocations component gains a new empty-state UI component with observer-based change detection (ResizeObserver and MutationObserver) to track rendering state. NearbyLocationsCardsWrapper adds early loading state rendering and simplified empty-data checks. ReviewsSection introduces a new exported interface (ReviewsSectionProps), expands props destructuring to include editing state (puck), and updates empty-state logic to consider query status. The main template file adds a transformProps function that performs data migration and resolution before rendering.

Sequence Diagram

sequenceDiagram
    participant Mount as Component Mount
    participant Observer as ResizeObserver/<br/>MutationObserver
    participant State as Component State
    participant Render as Render Logic

    Mount->>Observer: Setup watchers on mount<br/>(cards wrapper + empty-state)
    Observer->>Observer: Perform initial check
    Observer->>State: Detect changes in DOM
    
    Note over State: isEmptyState evaluation:<br/>1. Empty-state marker present?<br/>2. Element height > 0?<br/>3. Content presence (wrapper)?<br/>4. Loading state?
    
    State->>Render: isEmptyState = true
    Render->>Render: Render NearbyLocationsEmptyState<br/>(with MapPinOff icon + text)
    
    Note over Render: Alternative path
    
    State->>Render: isEmptyState = false
    Render->>Render: Render standard<br/>NearbyLocationsSection

Loading

Possibly related PRs

• feat: empty state for NearbyLocations #868: Modifies NearbyLocations UI to add editor empty-state rendering, including changes to NearbyLocations.tsx and related locale keys.
• feat: empty state for reviews section #869: Modifies ReviewsSection to add editor empty-state with new props/interface and status-based empty-state logic.
• fix: language picker when url templates are in use #840: Updates NearbyLocations to call resolvePageSetUrlTemplate, directly related to the same component being modified here.

Suggested reviewers

• briantstephan
• benlife5
• mkilpatrick

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch demo

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5fd1aae and cdb62b9.

📒 Files selected for processing (4)

• packages/visual-editor/src/components/pageSections/NearbyLocations/NearbyLocations.tsx (3 hunks)
• packages/visual-editor/src/components/pageSections/NearbyLocations/NearbyLocationsCardsWrapper.tsx (1 hunks)
• packages/visual-editor/src/components/pageSections/ReviewsSection/ReviewsSection.tsx (5 hunks)
• packages/visual-editor/src/vite-plugin/templates/main.tsx (1 hunks)

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}