A few more fixes

core/app/components/forms/AddRelatedPubsPanel.tsx

@@ -72,6 +72,7 @@ export const AddRelatedPubsPanel = ({
 			.map((selection) => selection[0] as PubsId);
 		const selectedPubs = pubs.filter((p) => selectedPubIds.includes(p.id));
 		onAdd(selectedPubs);
+		onCancel();
 	};
 
 	return (

core/app/components/forms/elements/RelatedPubsElement.tsx

@@ -85,7 +85,8 @@ export const InnerValue = ({
 	element,
 	...props
 }: PubFieldFormElementProps & { slug: string }) => {
-	const label = element.config.label || element.label || slug;
+	const configLabel = "label" in element.config ? element.config.label : undefined;
+	const label = configLabel || element.label || slug;
 
 	return <PubFieldFormElement {...props} element={element} slug={slug} label={label} />;
 };
@@ -121,26 +122,23 @@ export const RelatedPubsElement = ({
 		);
 	}, [pubs]);
 
+	const linkedPubs = fields.map((f) => f.relatedPubId);
+	const linkablePubs = pubs
+		// do not allow linking to itself or any pubs it is already linked to
+		.filter((p) => p.id !== pubId && !linkedPubs.includes(p.id));
+
 	return (
 		<>
 			<FormField
 				control={control}
 				name={slug}
 				render={({ field }) => {
-					const linkedPubs = Array.isArray(field.value)
-						? field.value.map((v: FieldValue) => v.relatedPubId)
-						: [];
-					const linkablePubs = pubs
-						// do not allow linking to itself or any pubs it is already linked to
-						.filter((p) => p.id !== pubId && !linkedPubs.includes(p.id));
-
 					const handleAddPubs = (newPubs: GetPubsResult) => {
 						const values = newPubs.map((p) => ({ relatedPubId: p.id, value: null }));
 						for (const value of values) {
 							append(value);
 						}
 					};
-
 					return (
 						<FormItem>
 							{showPanel && (
@@ -159,7 +157,7 @@ export const RelatedPubsElement = ({
 										disabled={!isEnabled}
 										onAdd={() => setShowPanel(true)}
 									>
-										{Array.isArray(field.value) ? (
+										{fields.length ? (
 											<div className="flex flex-col gap-2">
 												{fields.map((item, index) => {
 													const handleRemovePub = () => {

core/lib/fields/richText.ts

@@ -53,7 +53,7 @@ export const parseRichTextForPubFieldsAndRelatedPubs = <T extends JsonValue>({
 	pubId: PubsId;
 	values: Record<string, T>;
 }) => {
-	const newValues = structuredClone(values);
+	const newValues: Record<string, T | string> = structuredClone(values);
 	const pubs: PubCreate[] = [];
 
 	// Find a rich text value if one exists

packages/db/src/public/PublicSchema.ts

@@ -33,6 +33,10 @@ import type { StagesTable } from "./Stages";
 import type { UsersTable } from "./Users";
 
 export interface PublicSchema {
+	membership_capabilities: MembershipCapabilitiesTable;
+
+	pub_values_history: PubValuesHistoryTable;
+
 	_prisma_migrations: PrismaMigrationsTable;
 
 	users: UsersTable;
@@ -88,8 +92,4 @@ export interface PublicSchema {
 	stage_memberships: StageMembershipsTable;
 
 	form_memberships: FormMembershipsTable;
-
-	membership_capabilities: MembershipCapabilitiesTable;
-
-	pub_values_history: PubValuesHistoryTable;
 }