@the-portland-company/shell 0.37.0 → 0.38.0

package/CHANGELOG.md

@@ -1,3 +1,26 @@
+## 0.37.1 — 2026-07-27
+
+### Fixed — cross-app navigation kept the user's active organization
+
+Every link the Relationship tree emitted was bare: `Forms` / `Surveys` / `Polls`
+went to their standalone apps, and every core route went to `app.politogyvrm.com`,
+with no `?org=` selector. A multi-org user working in one organization clicked a
+feature and landed in whichever organization the destination resolved by default
+(reported: working in "We Need Steve", clicked Contacts, arrived in another org's
+contacts). Intra-app links already carried the selector, so the jump only
+happened when crossing apps.
+
+`standardPolitogyNav` now stamps `?org=<full-uuid>` on EVERY link it builds, in
+every tree — the Campaign and Petition trees already did. The full UUID is the
+only collision-free selector (contacts-api matches exact id first, then a short
+suffix that can collide across a user's orgs), and `isNavItemActive` already
+excludes `org` from active-detection, so nothing changes about which row is
+selected. The app-owned `contactsItem` override is still used verbatim; the
+`politogy_active_org` cookie remains the fallback for a bare URL.
+
+The same treatment applies to the chrome's own cross-app links — "Buy More Fuel"
+/ "Add fuel" and "View all" quarantined contacts, which all point into VRM core.
+
 ## 0.37.0 — 2026-07-27
 
 ### Fixed — organization logo renders in every app, not just VRM core

package/dist/app-chrome.cjs

@@ -150,6 +150,10 @@ var CORE_APP_ORIGIN = "https://app.politogyvrm.com";
 var CORE_BILLING_PATH = "/organization-settings/subscription-billing/text-email";
 var CORE_BILLING_URL = `${CORE_APP_ORIGIN}${CORE_BILLING_PATH}`;
 var QUARANTINED_CONTACTS_URL = `${CORE_APP_ORIGIN}/relationship/contacts/quarantined`;
+function withOrgParam(url, orgId) {
+  if (!orgId) return url;
+  return `${url}${url.includes("?") ? "&" : "?"}org=${orgId}`;
+}
 function resolveOrgLogoUrl(url) {
   const raw = url?.trim();
   if (!raw) return void 0;
@@ -504,7 +508,7 @@ function FuelRow({
     ) })
   ] });
 }
-function LowFuelHint({ lowFuel }) {
+function LowFuelHint({ lowFuel, orgId }) {
   const [dismissed, setDismissed] = react.useState(() => {
     if (typeof window === "undefined") return false;
     try {
@@ -548,7 +552,7 @@ function LowFuelHint({ lowFuel }) {
             /* @__PURE__ */ jsxRuntime.jsx(
               "a",
               {
-                href: CORE_BILLING_URL,
+                href: withOrgParam(CORE_BILLING_URL, orgId),
                 className: "mt-1 inline-block text-sm font-bold underline hover:opacity-90",
                 children: "Buy More Fuel"
               }
@@ -615,7 +619,7 @@ function FuelAction({ orgId }) {
                   label: "SMS Signal",
                   channel: fuel?.signal ?? null,
                   loading,
-                  addHref: CORE_BILLING_URL,
+                  addHref: withOrgParam(CORE_BILLING_URL, orgId),
                   addLabel: "Add SMS Signal fuel"
                 }
               ),
@@ -625,7 +629,7 @@ function FuelAction({ orgId }) {
                   label: "Email Broadcast",
                   channel: fuel?.broadcast ?? null,
                   loading,
-                  addHref: CORE_BILLING_URL,
+                  addHref: withOrgParam(CORE_BILLING_URL, orgId),
                   addLabel: "Add Email Broadcast fuel"
                 }
               )
@@ -633,7 +637,7 @@ function FuelAction({ orgId }) {
           ] })
         }
       ),
-      /* @__PURE__ */ jsxRuntime.jsx(LowFuelHint, { lowFuel: low })
+      /* @__PURE__ */ jsxRuntime.jsx(LowFuelHint, { lowFuel: low, orgId })
     ] })
   );
 }
@@ -928,7 +932,7 @@ function NotificationsAction({ orgId }) {
                   /* @__PURE__ */ jsxRuntime.jsx(
                     "a",
                     {
-                      href: QUARANTINED_CONTACTS_URL,
+                      href: withOrgParam(QUARANTINED_CONTACTS_URL, orgId),
                       className: "text-[11px] font-semibold text-[color:var(--color-race-red)] hover:underline",
                       children: "View all"
                     }
@@ -1396,15 +1400,15 @@ function standardPolitogyNav(opts) {
   const base = opts.coreBaseUrl.replace(/\/+$/, "");
   const abs = (path) => `${base}${path.startsWith("/") ? path : `/${path}`}`;
   const icon = (key) => opts.icons?.[key];
-  const leaves = (children) => children.map((c) => ({ label: c.label, icon: icon(c.icon), href: abs(c.path) }));
+  const orgParam = opts.activeOrgId ? `org=${opts.activeOrgId}` : "";
+  const withOrg = (url) => orgParam ? `${url}${url.includes("?") ? "&" : "?"}${orgParam}` : url;
+  const leaves = (children) => children.map((c) => ({ label: c.label, icon: icon(c.icon), href: withOrg(abs(c.path)) }));
   const branches = (children) => children.map((c) => ({
     label: c.label,
     icon: icon(c.icon),
-    href: abs(c.path),
+    href: withOrg(abs(c.path)),
     children: c.children ? leaves(c.children) : void 0
   }));
-  const orgParam = opts.activeOrgId ? `org=${opts.activeOrgId}` : "";
-  const withOrg = (url) => orgParam ? `${url}${url.includes("?") ? "&" : "?"}${orgParam}` : url;
   const buildRelationship = () => {
     const contacts = opts.contactsItem ? {
       label: "Contacts",
@@ -1414,22 +1418,22 @@ function standardPolitogyNav(opts) {
     } : {
       label: "Contacts",
       icon: icon("users"),
-      href: abs("/relationship/contacts/dashboard"),
+      href: withOrg(abs("/relationship/contacts/dashboard")),
       children: leaves(CONTACTS_DEFAULT_CHILDREN)
     };
     return [
-      { label: "Dashboard", icon: icon("home"), href: abs("/relationship/dashboard") },
+      { label: "Dashboard", icon: icon("home"), href: withOrg(abs("/relationship/dashboard")) },
       contacts,
       {
         label: "Email Broadcast",
         icon: icon("mail"),
-        href: abs("/relationship/email/dashboard"),
+        href: withOrg(abs("/relationship/email/dashboard")),
         children: leaves(EMAIL_CHILDREN)
       },
       {
         label: "SMS/MMS Blast",
         icon: icon("messageSquare"),
-        href: abs("/relationship/sms-mms-blast/dashboard"),
+        href: withOrg(abs("/relationship/sms-mms-blast/dashboard")),
         children: leaves(SMS_CHILDREN)
       },
       // These two carry no top-level href in VRM core (parent row only toggles).
@@ -1437,25 +1441,25 @@ function standardPolitogyNav(opts) {
       {
         label: "Forms",
         icon: icon("activity"),
-        href: "https://forms.politogyvrm.com/",
-        children: APP_FORMS_CHILDREN.map((c) => ({ label: c.label, icon: icon(c.icon), href: c.href }))
+        href: withOrg("https://forms.politogyvrm.com/"),
+        children: APP_FORMS_CHILDREN.map((c) => ({ label: c.label, icon: icon(c.icon), href: withOrg(c.href) }))
       },
       {
         label: "Surveys",
         icon: icon("fileText"),
-        href: "https://surveys.politogyvrm.com/surveys",
-        children: APP_SURVEYS_CHILDREN.map((c) => ({ label: c.label, icon: icon(c.icon), href: c.href }))
+        href: withOrg("https://surveys.politogyvrm.com/surveys"),
+        children: APP_SURVEYS_CHILDREN.map((c) => ({ label: c.label, icon: icon(c.icon), href: withOrg(c.href) }))
       },
       {
         label: "Polls",
         icon: icon("barChart"),
-        href: "https://polls.politogyvrm.com/",
-        children: APP_POLLS_CHILDREN.map((c) => ({ label: c.label, icon: icon(c.icon), href: c.href }))
+        href: withOrg("https://polls.politogyvrm.com/"),
+        children: APP_POLLS_CHILDREN.map((c) => ({ label: c.label, icon: icon(c.icon), href: withOrg(c.href) }))
       },
       {
         label: "Organization Settings",
         icon: icon("briefcase"),
-        href: abs("/organization-settings"),
+        href: withOrg(abs("/organization-settings")),
         children: branches(ORG_SETTINGS_CHILDREN)
       }
     ];
@@ -2442,11 +2446,13 @@ function NavBetaBanner({ text, linkLabel, href }) {
       role: "note",
       className: "w-full px-4 py-2 text-center text-sm",
       style: {
-        // Brand red fill, white text, no borders (HITL 2026-07-24). The
-        // --beta-banner-* tokens still win when the Brand sheet emits them;
-        // otherwise the banner takes the brand accent red.
-        background: "var(--beta-banner-bg, var(--color-accent, #E63946))",
-        color: "var(--beta-banner-text, #FFFFFF)"
+        // Amber warning fill, matching the "10DLC Registration Required" notice
+        // (HITL 2026-07-29). This used to fall through to `--color-accent`, the
+        // brand RED, so a participation notice read as an error. The
+        // --beta-banner-* tokens carry the amber defaults (chrome.css) and a
+        // Brand edit still wins over them.
+        background: "var(--beta-banner-bg, #FEEBC8)",
+        color: "var(--beta-banner-text, #1A202C)"
       },
       children: [
         /* @__PURE__ */ jsxRuntime.jsx("span", { children: body }),
@@ -2457,7 +2463,7 @@ function NavBetaBanner({ text, linkLabel, href }) {
             target: "_blank",
             rel: "noreferrer",
             className: "font-bold underline",
-            style: { color: "var(--beta-banner-text, #FFFFFF)" },
+            style: { color: "var(--beta-banner-text, #1A202C)" },
             children: label
           }
         )