Skip to content

Commit

Permalink
[Job Launcher][Client] Empty fields in escrow details. (#2687)
Browse files Browse the repository at this point in the history
* Fix empty description for CVAT jobs and change `Fortune's request` to `Task size` for non-fortune jobs

* Remove console log
  • Loading branch information
flopez7 authored Oct 24, 2024
1 parent f51ba67 commit 53899b8
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -163,13 +163,19 @@ export default function JobDetail() {
label="Chain Id"
value={data.manifest.chainId}
/>
<CardTextRow label="Title" value={data.manifest.title} />
{data.manifest.title && (
<CardTextRow label="Title" value={data.manifest.title} />
)}
<CardTextRow
label="Description"
value={data.manifest.description}
/>
<CardTextRow
label="Fortune's request"
label={
data.manifest.requestType === 'FORTUNE'
? "Fortune's request"
: 'Task size'
}
value={data.manifest.submissionsRequired}
/>
<CardTextRow
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1551,9 +1551,10 @@ export class JobService {
specificManifestDetails = {
requestType: manifest.annotation?.type,
submissionsRequired: manifest.annotation?.job_size,
...(manifest.annotation.qualifications &&
manifest.annotation.qualifications?.length > 0 && {
qualifications: manifest.annotation.qualifications,
description: manifest.annotation?.description,
...(manifest.annotation?.qualifications &&
manifest.annotation?.qualifications?.length > 0 && {
qualifications: manifest.annotation?.qualifications,
}),
};
}
Expand Down

0 comments on commit 53899b8

Please sign in to comment.