@propeller-commerce/propeller-v2-react-ui 0.16.0 → 0.18.0

package/CHANGELOG.md

@@ -8,6 +8,67 @@ once it reaches 1.0. Until then (the `0.x` line) the public API may change
 between minor versions; breaking changes are called out below and in
 [MIGRATION.md](./MIGRATION.md).
 
+## [0.18.0] - 2026-08-20
+
+### Added
+
+- **After-hooks now say WHAT happened, not just that something did.** Three
+  callbacks reported success with no argument, which made them unusable for
+  anything that has to distinguish the outcome — analytics, audit trails, or
+  optimistic UI. All three arguments are optional, so existing zero-argument
+  callbacks keep working unchanged.
+  - `AddToFavorite.onFavoriteChanged(change?)` — `{ action: 'added' | 'removed',
+    listId, productId, clusterId }`. It fired identically for an add and a
+    removal, so a host could not report "added to wishlist" without inventing
+    the direction, and a wishlist metric that also counts removals is worse
+    than no metric.
+  - `FavoriteLists.onListChanged(change?)` / `useFavorites` — `{ action:
+    'created' | 'updated' | 'deleted', listId, name, isDefault }`. Note the
+    existing `onCreate` / `onEdit` / `onDelete` props are *overrides* that
+    replace the default mutation, so they were never usable as notifications.
+  - `QuickOrder.onTemplateDownload()` — the template link was a bare `<a>`, so
+    a buyer fetching the spreadsheet left no trace. Navigation is untouched.
+
+### Notes
+
+- `QuoteActions` deliberately gains no `afterReject`: the component has no
+  reject action to hook — only accept. A rejection callback would be dead API.
+- `PurchaseAuthorizationRequests` needs no new prop either; the existing
+  `afterDeleteRequest` already covers the manager rejecting a request, which is
+  what "delete" means in that UI.
+
+## [0.17.0] - 2026-08-20
+
+### Fixed
+
+- **A host `className` override could not beat the component's own utility
+  (PWP-942 #17).** Class lists were built by string-appending the override
+  after the package's defaults, which decides nothing — the cascade does, not
+  attribute order. `iconClassName="text-cocoa"` on `AccountIconAndMenu` (baked-in
+  `text-white`) produced an element carrying both and rendered white, while the
+  same override on `CartIconAndSidebar` appeared to work only because its
+  default happened to lose anyway. Every override site now merges through
+  `tailwind-merge`, so a conflicting utility is replaced rather than raced.
+  59 sites across 47 components; BEM hooks and non-conflicting utilities are
+  untouched.
+
+- **Anonymous catalog listings were scoped differently on the client than on
+  the server (PWP-942 #22).** Listing hooks derived `userId` only from a
+  logged-in user and omitted the key entirely for guests, while a server-side
+  seed scopes anonymous queries to the channel's `anonymousUserId`. The two
+  therefore asked different questions, and the client refetch quietly replaced
+  a correctly-scoped product list with a differently-scoped one — assortment
+  rules, negative order lists in particular, are applied per user. All five
+  listing hooks (`useProductSearch` ×2, `useProductSlider`, `useQuickOrder`,
+  `useSpareParts`) now resolve the id through one shared helper. Hosts that
+  supply no `anonymousUserId` behave exactly as before.
+
+### Added
+
+- **`configuration.anonymousUserId`** — the channel's guest account, resolved
+  server-side and handed to the package so client-side listings can scope to
+  it. Same route `baseCategoryId` takes since PWP-913; no module guesses it.
+
 ## [0.16.0] - 2026-08-12
 
 ### Fixed

package/README.md

@@ -307,9 +307,17 @@ and pass it once.
 | `language`      | `string`                      | Locale code used to resolve localized content (e.g. `'NL'`, `'EN'`).        |
 | `includeTax`    | `boolean`                     | Whether displayed prices include tax.                                       |
 | `currency`      | `string`                      | Currency symbol for price formatting. Default: `'€'`.                       |
-| `configuration` | `unknown`                     | Free-form config bag forwarded to components — stuff your own settings in.  |
+| `configuration` | `unknown`                     | Config bag forwarded to components. Recognized keys: `baseCategoryId`, `anonymousUserId` (see below), image filters — plus anything of your own. |
 | `portalMode`    | `string`                      | Storefront mode (e.g. `'open'`, `'closed'`).                                |
 
+**`configuration.anonymousUserId`** — the channel's guest account. Listing
+hooks scope logged-out catalog queries to it, so the client asks the same
+question your server-rendered seed did. Resolve it server-side (only the
+server can reach the channel) and pass it in; when it's absent, anonymous
+queries send no `userId` and the backend falls back to the api key's own
+scope. Getting this wrong is invisible — the grid renders, just with the
+wrong assortment.
+
 The `value` object is reactive — when your auth/company/language state
 changes, recompute it (memoize on those dependencies) and the provider
 propagates the new value. Components re-render with fresh infra; service
@@ -584,15 +592,19 @@ cascade order — no `!important` needed:
 .propeller-breadcrumbs__separator { display: none; }
 ```
 
-**3. Per-instance `className`** — every component appends `props.className`
-on its root, so a one-off override is a regular prop:
+**3. Per-instance `className`** — every component merges `props.className`
+into its root classes, so a one-off override is a regular prop:
 
 ```tsx
 <ProductCard product={p} className="ring-2 ring-yellow-400" />
 ```
 
-`className` **adds to** the base classes — it doesn't replace them. To strip
-a default, use a BEM hook.
+The merge runs through [`tailwind-merge`](https://github.com/dcastil/tailwind-merge)
+(since 0.17.0), so an override **replaces** the base utility it conflicts with
+rather than racing it in the cascade — `iconClassName="text-cocoa"` on a
+component whose default is `text-white` now renders cocoa. Non-conflicting
+utilities and the BEM hooks are left untouched. To remove a default outright
+(rather than override it), use a BEM hook.
 
 See [STYLING.md](./STYLING.md) for the full token list, the complete BEM
 hook catalog, and the cascade rationale.