@aglyn/plugins-marketing 1.0.0-beta.173 → 1.0.0-beta.175

package/README.md

@@ -19,10 +19,11 @@ The plugin declares four registrars in `plugins.config.json`, plus a `site` modu
 **Console** (`registerMarketingConsole`, the `console` registrar):
 
 - A `Marketing` nav item at `/marketing` with the sections Overview, Campaigns, Conversions, Overlays and A/B testing. Each section is a route; the page is code-split.
+- The same page at the organization level, `/[orgSlug]/marketing`, declared in `orgNavItems` with the sections Campaigns and Emails. Campaigns and their sends belong to the organization (`orgs/{orgId}/emailCampaigns`, `orgs/{orgId}/campaigns`): the org hub lists every one, places a campaign on every site or on chosen sites (`visibleTo`), and asks which site an email is sent as before writing it. A site's hub lists the campaigns placed on that site and the emails sent as it. Both levels share the `release_marketing` flag.
 - Campaigns end to end. The plugin owns the campaign composer, test sends, scheduling, reports and recipients. The individual messages are listed on the Emails page, but that page belongs to the email plugin, which hosts an `emailMessages` zone; this plugin fills it with the message list, one message's page and its composer. It also fills `emailTemplateRecipients` (who received the sends built from a template).
 - Widgets in other hosts' slots: the "Last campaign" card on the site dashboard (`hostDashboard`), the campaigns card in the Inbox (`inboxCampaigns`), and campaign attribution on a record's page (`crmRecordAttribution`, `inboxRecordAttribution`).
 - Zones this plugin hosts for other plugins to fill, registered with `registerPluginZone`: two on the A/B testing card (beside the variants, and below a test's results), and five bare zones in the campaign composer and on a message's page for whichever plugin keeps the mail itself: the topic picker, the topic options, the add-a-sender editor, the design creator and the sent-design preview.
-- A record route for `campaign`, so another plugin can link to a campaign without knowing this plugin's URLs.
+- A record route for `campaign`, at both the site and the organization level, so another plugin can link to a campaign without knowing this plugin's URLs.
 
 **Server**, exported from `@aglyn/plugins-marketing/server`:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aglyn/plugins-marketing",
-  "version": "1.0.0-beta.173",
+  "version": "1.0.0-beta.175",
   "license": "Apache-2.0",
   "homepage": "https://aglyn.com",
   "repository": {
@@ -25,20 +25,20 @@
     "./package.json": "./package.json"
   },
   "dependencies": {
-    "@aglyn/aglyn": "1.0.0-beta.173",
-    "@aglyn/shared-data-enums": "1.0.0-beta.173",
-    "@aglyn/shared-data-mdi": "1.0.0-beta.173",
-    "@aglyn/shared-ui-email-campaigns": "1.0.0-beta.173",
-    "@aglyn/shared-ui-jsx": "1.0.0-beta.173",
-    "@aglyn/shared-ui-jsx-forms": "1.0.0-beta.173",
-    "@aglyn/shared-ui-next": "1.0.0-beta.173",
-    "@aglyn/shared-ui-snackstack": "1.0.0-beta.173",
-    "@aglyn/shared-util-email": "1.0.0-beta.173",
-    "@aglyn/shared-util-http": "1.0.0-beta.173",
-    "@aglyn/shared-util-timestamp": "1.0.0-beta.173",
-    "@aglyn/tenant-data-admin": "1.0.0-beta.173",
-    "@aglyn/tenant-feature-instance": "1.0.0-beta.173",
-    "@aglyn/tenant-runtime": "1.0.0-beta.173",
+    "@aglyn/aglyn": "1.0.0-beta.175",
+    "@aglyn/shared-data-enums": "1.0.0-beta.175",
+    "@aglyn/shared-data-mdi": "1.0.0-beta.175",
+    "@aglyn/shared-ui-email-campaigns": "1.0.0-beta.175",
+    "@aglyn/shared-ui-jsx": "1.0.0-beta.175",
+    "@aglyn/shared-ui-jsx-forms": "1.0.0-beta.175",
+    "@aglyn/shared-ui-next": "1.0.0-beta.175",
+    "@aglyn/shared-ui-snackstack": "1.0.0-beta.175",
+    "@aglyn/shared-util-email": "1.0.0-beta.175",
+    "@aglyn/shared-util-http": "1.0.0-beta.175",
+    "@aglyn/shared-util-timestamp": "1.0.0-beta.175",
+    "@aglyn/tenant-data-admin": "1.0.0-beta.175",
+    "@aglyn/tenant-feature-instance": "1.0.0-beta.175",
+    "@aglyn/tenant-runtime": "1.0.0-beta.175",
     "@swc/helpers": "0.5.23"
   },
   "peerDependencies": {

package/src/lib/components/campaign-detail-card.d.ts

@@ -1,5 +1,6 @@
 export interface CampaignDetailCardProps {
-    hostId: string;
+    /** The site, or `null` on the org Marketing hub. */
+    hostId: string | null;
     /** A campaign container id, or a send id from before containers existed. */
     campaignId: string;
     /** The marketing hub URL, for the way back to the campaigns list. */
@@ -47,6 +48,16 @@ export interface CampaignDetailCardProps {
  *
  * The extra read is one document, and it buys the guarantee that a pasted
  * report link resolves whichever kind of id it carries.
+ *
+ * ## Under a site, and over the organization
+ *
+ * The container and its sends are the org's, so the same page renders on the
+ * org hub with `hostId` null. What changes there is everything that is a SITE
+ * fact: the emails listed are every site's rather than the ones sent as this
+ * site, the forms and screens filed under the campaign are gathered from each
+ * site it is placed on, the conversions are one chosen site's (they are
+ * recorded per site), and writing an email first asks which site it is sent
+ * as — its sender, its designs and its consent all belong to that site.
  */
 export declare function CampaignDetailCard(props: CampaignDetailCardProps): import("react").JSX.Element;
 export declare namespace CampaignDetailCard {

package/src/lib/components/campaign-detail-card.js

@@ -24,18 +24,20 @@ import RowActionsMenu from "@aglyn/shared-ui-jsx/components/row-actions-menu.com
 import { TABLE_ROW_HEIGHT } from "@aglyn/shared-ui-jsx/const/table-pagination";
 import { ceilingedWindow } from "@aglyn/tenant-feature-instance/hooks/host-collection-queries";
 import { PageHeaderRecord, pluginDocsHelp } from "@aglyn/aglyn";
-import { Alert, Box, Button, Chip, Divider, Stack, Typography } from "@mui/material";
-import { collection, deleteField, doc, documentId, limit, orderBy, query, updateDoc, where } from "firebase/firestore";
+import { Alert, Box, Button, Chip, Divider, MenuItem, Stack, TextField, Typography } from "@mui/material";
+import { collection, deleteField, documentId, limit, orderBy, query, updateDoc, where } from "firebase/firestore";
 import { useCallback, useMemo, useState } from "react";
 import { useRouter } from "next/navigation";
 import { useSnackbar } from "@aglyn/shared-ui-snackstack";
-import { useFirestore, useFirestoreCollection, useFirestoreDoc, useOrgDataScope } from "@aglyn/tenant-feature-instance";
-import { campaignRollup, campaignSendDisplay, campaignWindowState, emailListTimeMs } from "@aglyn/shared-ui-email-campaigns/model";
+import { useFirestore, useFirestoreCollection, useFirestoreDoc } from "@aglyn/tenant-feature-instance";
+import { campaignRollup, campaignSendDisplay, campaignSiteIds, campaignVisibleTo, campaignWindowState, emailListTimeMs } from "@aglyn/shared-ui-email-campaigns/model";
 import CampaignComposer from "./campaign-composer.js";
 import CampaignEditDrawer from "./campaign-edit-drawer.js";
 import { CampaignConversionsSection, CampaignDestinationsSection, CampaignRevenueSection, CampaignSequencesSection } from "./campaign-reach-sections.js";
 import CampaignMembersSection from "./campaign-members-section.js";
 import CampaignReportCard from "./campaign-report-card.js";
+import { campaignContainerDoc, campaignSendsQuery } from "./campaign-queries.js";
+import { orgSiteHubPath, orgSiteName, orgSiteOptions, topicCatalogHostId, useMarketingOrgId, useMarketingOrgMount } from "./marketing-org-mount.js";
 import { useCampaignManageApi } from "./use-campaign-send-api.js";
 import { useEmailsHubPath } from "./use-emails-hub-path.js";
 import { useCampaignTopicOptions } from "./use-campaign-topic-options.js";
@@ -98,28 +100,44 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
  *
  * The extra read is one document, and it buys the guarantee that a pasted
  * report link resolves whichever kind of id it carries.
+ *
+ * ## Under a site, and over the organization
+ *
+ * The container and its sends are the org's, so the same page renders on the
+ * org hub with `hostId` null. What changes there is everything that is a SITE
+ * fact: the emails listed are every site's rather than the ones sent as this
+ * site, the forms and screens filed under the campaign are gathered from each
+ * site it is placed on, the conversions are one chosen site's (they are
+ * recorded per site), and writing an email first asks which site it is sent
+ * as — its sender, its designs and its consent all belong to that site.
  */ export function CampaignDetailCard(props) {
     var _campaign_listIds;
+    var _placedSites_;
     const { hostId, campaignId, basePath } = props;
     const firestore = useFirestore();
     // The sibling hub, for the two records this page links to but does not own:
     // each message's report and the template it was built from.
     const emailsHub = useEmailsHubPath();
     const router = useRouter();
-    const { scope: dataScope } = useOrgDataScope({
-        hostId
-    });
+    const orgMount = useMarketingOrgMount();
+    const { orgId } = useMarketingOrgId(hostId);
     const { confirm } = useConfirmationContext();
     const { enqueueSnackbar } = useSnackbar();
-    const manageApi = useCampaignManageApi(hostId);
+    const manageApi = useCampaignManageApi(hostId, orgId);
+    /*
+   * The site an email written here is sent AS. Under a site it is that site
+   * and never asked; on the org hub it is chosen before the composer opens.
+   * Empty means nobody has chosen yet.
+   */ const [composeHostId, setComposeHostId] = useState('');
+    /** The site whose conversions the org hub is showing; empty until chosen. */ const [conversionsHostId, setConversionsHostId] = useState('');
     const [composing, setComposing] = useState(false);
     const [editing, setEditing] = useState(false);
     const [saving, setSaving] = useState(false);
     const [saveError, setSaveError] = useState(null);
     const [deleting, setDeleting] = useState(false);
-    const { data: campaign, status } = useFirestoreDoc(()=>doc(firestore, 'hosts', hostId, 'emailCampaigns', campaignId), [
+    const { data: campaign, status } = useFirestoreDoc(()=>orgId ? campaignContainerDoc(firestore, orgId, campaignId) : null, [
         firestore,
-        hostId,
+        orgId,
         campaignId
     ]);
     /*
@@ -133,8 +151,11 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
    * to avoid. Not listening is what a null builder means.
    *
    * The campaign's emails come back by the field each send carries: equality
-   * on one field ordered by DOCUMENT NAME, which Firestore's automatic
-   * single-field index serves without a composite one. Ordering on a date
+   * on one field ordered by DOCUMENT NAME. On the org hub that is all the
+   * query says, and the automatic single-field index serves it. Under a site
+   * the sends are also narrowed to the ones sent as that site — the scope
+   * filter that makes the read provable for a site collaborator — and that
+   * pair is the one composite index this page needs. Ordering on a date
    * would need that index AND would drop every send missing that particular
    * date — a sent send carries `sentAt`, a scheduled one `sendAtMs`, and
    * neither is on both. Sorted by date below, over a window this page holds
@@ -142,17 +163,21 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
    *
    * One document past the ceiling, so "this campaign has more emails than are
    * listed" is a fact rather than a guess.
-   */ const { data: sendDocs } = useFirestoreCollection(()=>campaign ? query(collection(firestore, 'hosts', hostId, 'campaigns'), where('emailCampaignId', '==', campaignId), orderBy(documentId()), limit(CAMPAIGN_EMAIL_CEILING + 1)) : null, [
+   */ const { data: sendDocs } = useFirestoreCollection(()=>{
+        const sends = campaign ? campaignSendsQuery(firestore, orgId, hostId) : null;
+        return sends ? query(sends, where('emailCampaignId', '==', campaignId), orderBy(documentId()), limit(CAMPAIGN_EMAIL_CEILING + 1)) : null;
+    }, [
         firestore,
+        orgId,
         hostId,
         campaignId,
         Boolean(campaign)
     ], {
         idField: '$id'
     });
-    const { data: listDocs } = useFirestoreCollection(()=>campaign && dataScope ? query(collection(firestore, dataScope[0], dataScope[1], 'lists'), limit(50)) : null, [
+    const { data: listDocs } = useFirestoreCollection(()=>campaign && orgId ? query(collection(firestore, 'orgs', orgId, 'lists'), limit(50)) : null, [
         firestore,
-        dataScope,
+        orgId,
         Boolean(campaign)
     ], {
         idField: '$id'
@@ -165,7 +190,7 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
    * campaign's numbers for a field they are not looking at. The composer
    * carries its own read of the same catalog, behind its own button, for the
    * same reason.
-   */ const { topics, source: topicSource } = useCampaignTopicOptions(hostId, {
+   */ const { topics, source: topicSource } = useCampaignTopicOptions(topicCatalogHostId(hostId, orgMount), {
         enabled: editing
     });
     const topicOptions = useMemo(()=>topics.map((topic)=>({
@@ -213,18 +238,25 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
    * A cleared date is written as `null` — the absence the model already
    * spells — and a cleared topic REMOVES the field, because the model has no
    * null topic and `campaignTopicId` is handed to the composer as one.
+   *
+   * `visibleTo` is written only from the org hub, and only when its drawer
+   * says the placement changed. A site hub never touches it: the rules let a
+   * site collaborator edit a campaign only while its scope is unchanged, and
+   * a site hub has no field for it anyway.
    *=========================================*/ const handleSave = useCallback(async (values)=>{
-        if (saving) return;
+        if (saving || !orgId) return;
         setSaving(true);
         setSaveError(null);
         try {
-            await updateDoc(doc(firestore, 'hosts', hostId, 'emailCampaigns', campaignId), {
+            await updateDoc(campaignContainerDoc(firestore, orgId, campaignId), _extends({
                 name: values.name,
                 startAtMs: values.startAtMs,
                 endAtMs: values.endAtMs,
                 listIds: values.listIds,
                 topicId: values.topicId ? values.topicId : deleteField()
-            });
+            }, !hostId && values.siteIds !== undefined ? {
+                visibleTo: campaignVisibleTo(values.siteIds)
+            } : {}));
             setEditing(false);
             enqueueSnackbar('Campaign updated', {
                 variant: 'success',
@@ -241,6 +273,7 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
         enqueueSnackbar,
         firestore,
         hostId,
+        orgId,
         saving
     ]);
     /*==========================================
@@ -313,8 +346,9 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
         sends.length
     ]);
     // Still settling. Falling through to the send report here would flash "this
-    // campaign could not be loaded" on every open of a campaign that exists.
-    if (!campaign && status === 'loading') return null;
+    // campaign could not be loaded" on every open of a campaign that exists —
+    // and under a site nothing is read at all until the site's org is known.
+    if (!campaign && (status === 'loading' || !orgId)) return null;
     /*
    * NOT A CONTAINER — so it is a send id, and the send's own report is what
    * this URL has always meant.
@@ -352,7 +386,16 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
    * Nor is a redirect the answer to the second. A redirect would be another
    * thing to be wrong about an id that is inside an HMAC; a page that keeps
    * working has nothing to get wrong, and it costs nothing to leave standing.
-   *=========================================*/ const sendHref = (send)=>emailsHub ? `${emailsHub}/messages/${send.$id}` : undefined;
+   *=========================================*/ /*
+   * On the org hub there is no Emails console to send a reader to, so a
+   * message opens on the org hub's own Emails section — the same page, over
+   * every site.
+   */ const messagesPath = orgMount ? `${orgMount.basePath}/emails` : emailsHub ? `${emailsHub}/messages` : null;
+    const sendHref = (send)=>messagesPath ? `${messagesPath}/${send.$id}` : undefined;
+    /** A template is a site's design, so it opens on the site the send used. */ const templatesPath = (send)=>{
+        const hub = orgMount ? orgSiteHubPath(orgMount, send.hostId, 'emails') : emailsHub;
+        return hub ? `${hub}/templates` : null;
+    };
     /**
    * What one of this campaign's emails can be opened into.
    *
@@ -367,6 +410,7 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
    */ const sendActions = (send)=>{
         var _send_templateScreenId;
         const templateScreenId = String((_send_templateScreenId = send.templateScreenId) != null ? _send_templateScreenId : '');
+        const templates = templatesPath(send);
         return [
             {
                 key: 'details',
@@ -386,8 +430,8 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
                     path: mdiPaletteOutline.path,
                     size: 0.8
                 }),
-                href: templateScreenId && emailsHub ? `${emailsHub}/templates/${templateScreenId}` : undefined,
-                disabled: !templateScreenId || !emailsHub,
+                href: templateScreenId && templates ? `${templates}/${templateScreenId}` : undefined,
+                disabled: !templateScreenId || !templates,
                 disabledReason: templateScreenId ? 'This site’s console URL has not resolved yet' : 'This message was not built from a template'
             }
         ];
@@ -504,6 +548,17 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
     const windowState = campaignWindowState(campaign, Date.now());
     const start = campaign.startAtMs ? new Date(campaign.startAtMs).toLocaleDateString() : '';
     const end = campaign.endAtMs ? new Date(campaign.endAtMs).toLocaleDateString() : '';
+    /*
+   * THE SITES IT IS PLACED ON, which the org hub needs three times over: the
+   * forms and screens filed under it are gathered from each of them, its
+   * conversions are read for one of them, and an email written here is sent
+   * as one of them. Every site when it is placed everywhere. Under a site the
+   * answer is that site, and none of this is read.
+   */ const placedIds = campaignSiteIds(campaign);
+    const placedSites = orgMount ? orgMount.hosts.filter((site)=>placedIds === null || placedIds.includes(site.id)) : [];
+    const conversionsHost = hostId != null ? hostId : conversionsHostId || ((_placedSites_ = placedSites[0]) == null ? void 0 : _placedSites_.id) || '';
+    const conversionsBasePath = hostId ? basePath : orgSiteHubPath(orgMount, conversionsHost, 'marketing');
+    /** Asked only when there is a choice to make; one site answers itself. */ const writeAsHostId = hostId != null ? hostId : placedSites.length === 1 ? placedSites[0].id : composeHostId;
     /* The card, named so the page chrome above it is a plain list of
      what this surface publishes upward. */ const card = /*#__PURE__*/ _jsxs(CardDisplay, {
         header: 'Campaign',
@@ -600,37 +655,71 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
                             })
                         ]
                     }),
+                    orgMount ? /*#__PURE__*/ _jsx(Typography, {
+                        variant: "body2",
+                        color: "text.secondary",
+                        children: placedIds === null ? 'Offered on every site' : placedIds.length ? `Offered on ${placedIds.map((id)=>orgSiteName(orgMount, id)).join(', ')}` : 'Not offered on any site yet — edit the campaign to place it'
+                    }) : null,
                     /*#__PURE__*/ _jsx(Divider, {}),
-                    /*#__PURE__*/ _jsx(CampaignConversionsSection, {
-                        hostId: hostId,
+                    orgMount && placedSites.length > 1 ? /*#__PURE__*/ _jsx(TextField, {
+                        select: true,
+                        size: "small",
+                        label: "Conversions on",
+                        value: conversionsHost,
+                        onChange: (event)=>setConversionsHostId(event.target.value),
+                        sx: {
+                            maxWidth: 320
+                        },
+                        children: placedSites.map((site)=>/*#__PURE__*/ _jsx(MenuItem, {
+                                value: site.id,
+                                children: site.name || site.subdomain || site.id
+                            }, site.id))
+                    }) : null,
+                    conversionsHost ? /*#__PURE__*/ _jsx(CampaignConversionsSection, {
+                        hostId: conversionsHost,
                         sendIds: sendIds,
                         truncated: sendsTruncated,
-                        basePath: basePath
+                        basePath: conversionsBasePath
+                    }, conversionsHost) : /*#__PURE__*/ _jsx(Typography, {
+                        variant: "body2",
+                        color: "text.secondary",
+                        children: 'Conversions are counted per site, and this campaign is not ' + 'placed on one yet.'
                     }),
                     /*#__PURE__*/ _jsx(Divider, {}),
                     /*#__PURE__*/ _jsx(CampaignSequencesSection, {
-                        hostId: hostId,
+                        orgId: orgId,
                         campaignId: campaignId
                     }),
                     /*#__PURE__*/ _jsx(Divider, {}),
                     /*#__PURE__*/ _jsx(CampaignRevenueSection, {
-                        hostId: hostId,
+                        orgId: orgId,
                         sendIds: sendIds,
                         truncated: sendsTruncated
                     }),
                     /*#__PURE__*/ _jsx(Divider, {}),
                     /*#__PURE__*/ _jsx(CampaignDestinationsSection, {
-                        hostId: hostId,
+                        orgId: orgId,
                         sendIds: sendIds,
                         truncated: sendsTruncated
                     }),
                     /*#__PURE__*/ _jsx(Divider, {}),
-                    /*#__PURE__*/ _jsx(CampaignMembersSection, {
+                    hostId ? /*#__PURE__*/ _jsx(CampaignMembersSection, {
                         hostId: hostId,
                         campaignId: campaignId,
                         startAtMs: campaign.startAtMs,
                         endAtMs: campaign.endAtMs
-                    }),
+                    }) : /*
+           * Forms and screens are a site's records, so on the org hub each
+           * site the campaign is placed on answers for its own. One section
+           * per site keeps each one's figures and ceilings its own rather
+           * than merging lists that were bounded separately.
+           */ placedSites.map((site)=>/*#__PURE__*/ _jsx(CampaignMembersSection, {
+                            hostId: site.id,
+                            siteName: site.name || site.subdomain || site.id,
+                            campaignId: campaignId,
+                            startAtMs: campaign.startAtMs,
+                            endAtMs: campaign.endAtMs
+                        }, site.id)),
                     /*#__PURE__*/ _jsx(Divider, {}),
                     /*#__PURE__*/ _jsx(Section, {
                         title: "The mail, across this campaign",
@@ -731,13 +820,34 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
                                     mb: 2
                                 }
                             }),
-                            /*#__PURE__*/ _jsx(CampaignComposer, {
-                                hostId: hostId,
+                            !hostId && placedSites.length > 1 ? /*#__PURE__*/ _jsx(TextField, {
+                                select: true,
+                                size: "small",
+                                label: "Send as",
+                                value: composeHostId,
+                                onChange: (event)=>setComposeHostId(event.target.value),
+                                helperText: "The site this email is sent from — its sender, designs and unsubscribe page",
+                                sx: {
+                                    mb: 2,
+                                    maxWidth: 420
+                                },
+                                fullWidth: true,
+                                children: placedSites.map((site)=>/*#__PURE__*/ _jsx(MenuItem, {
+                                        value: site.id,
+                                        children: site.name || site.subdomain || site.id
+                                    }, site.id))
+                            }) : null,
+                            writeAsHostId ? /*#__PURE__*/ _jsx(CampaignComposer, {
+                                hostId: writeAsHostId,
                                 emailCampaignId: campaignId,
                                 campaignListIds: listIds,
                                 campaignTopicId: campaign.topicId,
                                 onSent: ()=>setComposing(false)
-                            })
+                            }, writeAsHostId) : !placedSites.length && !hostId ? /*#__PURE__*/ _jsx(Typography, {
+                                variant: "body2",
+                                color: "text.secondary",
+                                children: 'Place this campaign on a site before writing an email in ' + 'it — every email is sent as one site.'
+                            }) : null
                         ]
                     }) : null
                 ]
@@ -755,6 +865,7 @@ const detailDocsHelp = pluginDocsHelp('emailCampaigns', {
                     };
                 }),
                 topics: topicOptions,
+                sites: orgMount ? orgSiteOptions(orgMount) : undefined,
                 busy: saving,
                 error: saveError,
                 onSubmit: (values)=>void handleSave(values)