Skip to content

Commit b44002a

Browse files
committed
fix: update PreviewData component to handle potential null values and improve layout consistency in Contributors page
1 parent 9d4dfad commit b44002a

File tree

2 files changed

+8
-6
lines changed
  • app/[locale]/dashboard/[entityType]/[entitySlug]

2 files changed

+8
-6
lines changed

app/[locale]/dashboard/[entityType]/[entitySlug]/dataset/[id]/edit/resources/components/PreviewData.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ const PreviewData = ({ isPreview, previewData }: EditProps) => {
1515
header: column,
1616
cell: ({ cell }: any) => {
1717
const value = cell.getValue();
18-
return <span>{value !== null ? value.toString() : 'N/A'}</span>;
18+
return <span>{value !== null ? value?.toString() : 'N/A'}</span>;
1919
},
2020
})) || [];
2121

app/[locale]/dashboard/[entityType]/[entitySlug]/usecases/edit/[id]/contributors/page.tsx

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,6 @@ const Details = () => {
204204
setStatus(loadingStates.some(Boolean) ? 'loading' : 'success');
205205
}, loadingStates);
206206

207-
208207
return (
209208
<div>
210209
{Users?.isLoading || allEntityDetails?.organizations?.length === 0 ? (
@@ -249,7 +248,10 @@ const Details = () => {
249248
</div>
250249
<div className="mt-3 flex flex-wrap gap-3 lg:mt-0">
251250
{formData.contributors.map((item) => (
252-
<div key={item.value} className='flex flex-col gap-2 items-center'>
251+
<div
252+
key={item.value}
253+
className="flex flex-col items-center gap-2"
254+
>
253255
<Image
254256
src={
255257
UseCaseData.data.useCases[0]?.contributors?.find(
@@ -283,7 +285,7 @@ const Details = () => {
283285
}}
284286
kind="tertiary"
285287
>
286-
<div className="flex items-center gap-2 rounded-2 p-2 bg-greyExtralight ">
288+
<div className="flex items-center gap-2 rounded-2 bg-greyExtralight p-2 ">
287289
<Text>{item.label}</Text>
288290
<Icon source={Icons.cross} size={18} />
289291
</div>
@@ -299,7 +301,7 @@ const Details = () => {
299301
title="SUPPORTED BY"
300302
label="Add Supporters"
301303
placeholder="Add Supporters"
302-
data={UseCaseData.data.useCases[0].supportingOrganizations}
304+
data={UseCaseData?.data?.useCases[0]?.supportingOrganizations}
303305
options={(allEntityDetails?.organizations || [])?.map(
304306
(org: any) => ({
305307
label: org.name,
@@ -340,7 +342,7 @@ const Details = () => {
340342
title="PARTNERED BY"
341343
label="Add Partners"
342344
placeholder="Add Partners"
343-
data={UseCaseData.data.useCases[0].partnerOrganizations}
345+
data={UseCaseData?.data?.useCases[0]?.partnerOrganizations}
344346
options={(allEntityDetails?.organizations || [])?.map(
345347
(org: any) => ({
346348
label: org.name,

0 commit comments

Comments
 (0)