Skip to content

Commit 2461357

Browse files
committed
feat: Add optional chaining to meetingTopics, leaderboard, and learningPoints in SummaryAgendaItems; update workgroup display in SummaryTemplate
1 parent 74f41fe commit 2461357

File tree

2 files changed

+8
-10
lines changed

2 files changed

+8
-10
lines changed

components/SummaryAgendaItems.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ const SummaryAgendaItems = ({ onUpdate }: any) => {
210210
<>
211211
<h3>Meeting Topics</h3>
212212
<div className={styles['discussion-points']}>
213-
{item.meetingTopics.map((topic: any, idx: number) => (
213+
{item.meetingTopics?.map((topic: any, idx: number) => (
214214
<Item
215215
key={`${agendaIndex}-meetingtopic-${idx}`}
216216
type="meetingTopics"
@@ -372,7 +372,7 @@ const SummaryAgendaItems = ({ onUpdate }: any) => {
372372
<>
373373
<h3>Leaderboard</h3>
374374
<div className={styles['discussion-points']}>
375-
{item.leaderboard.map((lb: any, lbIdx: number) => (
375+
{item.leaderboard?.map((lb: any, lbIdx: number) => (
376376
<Item
377377
key={`${agendaIndex}-leaderboard-${lbIdx}`}
378378
type="leaderboard"
@@ -403,7 +403,7 @@ const SummaryAgendaItems = ({ onUpdate }: any) => {
403403
<>
404404
<h3>Learning Points</h3>
405405
<div className={styles['discussion-points']}>
406-
{item.learningPoints.map((lp: any, lpIdx: number) => (
406+
{item.learningPoints?.map((lp: any, lpIdx: number) => (
407407
<Item
408408
key={`${agendaIndex}-learning-${lpIdx}`}
409409
type="learningPoints"

components/SummaryTemplate.tsx

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -106,15 +106,13 @@ const SummaryTemplate = ({ updateMeetings }: SummaryTemplateProps) => {
106106
// Only set local formData from context once:
107107
// ------------------------------------------
108108
useEffect(() => {
109-
if (!initializedRef.current && myVariable.summary) {
110-
initializedRef.current = true;
109+
// Only update if the workgroup name has actually changed
110+
if (myVariable.summary && myVariable.summary.workgroup !== formData.workgroup) {
111111
const filtered = filterFormData(myVariable.summary);
112112
setFormData(filtered);
113-
if (filtered.tags) {
114-
setTags(filtered.tags);
115-
}
113+
setTags(filtered.tags || {});
116114
}
117-
}, [myVariable.summary]);
115+
}, [myVariable.summary]);
118116

119117
// Keep formData.tags in sync with local tags:
120118
useEffect(() => {
@@ -269,7 +267,7 @@ const SummaryTemplate = ({ updateMeetings }: SummaryTemplateProps) => {
269267
)}
270268
{!loading && (
271269
<div className={styles['form-container']}>
272-
<h2>{formData.workgroup} {formData.meetingInfo.date}</h2>
270+
<h2>{myVariable.summary?.workgroup} {formData.meetingInfo.date}</h2>
273271
<div className={styles['gitbook-form']}>
274272
{/* Meeting Info */}
275273
{formData.meetingInfo.name && (

0 commit comments

Comments
 (0)