Skip to content

Commit 6a35dae

Browse files
Fix extra fields on findings not saving
The extra field was being set in the wrong place in the gql query data.
1 parent 37f05c2 commit 6a35dae

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

javascript/src/collab_server/handlers/finding.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ const FindingHandler = simpleModelHandler(
8585
(doc, id) => {
8686
const plainFields = doc.get("plain_fields", Y.Map<any>);
8787
const extraFields = extraFieldsFromYdoc(doc);
88+
console.log("DEBUG", extraFields);
8889
return {
8990
id,
9091
set: {
@@ -110,9 +111,9 @@ const FindingHandler = simpleModelHandler(
110111
findingGuidance: yjsToHtml(
111112
doc.get("findingGuidance", Y.XmlFragment)
112113
),
114+
extraFields,
113115
},
114116
tags: yjsToTags(doc.get("tags", Y.Map<boolean>)),
115-
extraFields,
116117
};
117118
}
118119
);

javascript/src/collab_server/handlers/report_finding_link.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,9 +118,9 @@ const ReportFindingLinkHandler = simpleModelHandler(
118118
affectedEntities: yjsToHtml(
119119
doc.get("affectedEntities", Y.XmlFragment)
120120
),
121+
extraFields,
121122
},
122123
tags: yjsToTags(doc.get("tags", Y.Map<boolean>)),
123-
extraFields,
124124
};
125125
}
126126
);

0 commit comments

Comments
 (0)