Skip to content

Commit a339cfa

Browse files
committed
deposit-ui: show icon and tooltip for new error format with severity error
1 parent 9d8e7a5 commit a339cfa

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

invenio_rdm_records/assets/semantic-ui/js/invenio_rdm_records/src/deposit/api/DepositRecordSerializer.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -369,7 +369,7 @@ export class RDMDepositRecordSerializer extends DepositRecordSerializer {
369369
// Form/Error UX is tackled in next sprint and this is good
370370
// enough for now.
371371
for (const e of errors) {
372-
if ("severity" in e && e.severity !== "error") {
372+
if ("severity" in e && "severity" in e) {
373373
// New error format with severity and description
374374
_set(deserializedErrors, e.field, {
375375
message: e.messages.join(" "),

invenio_rdm_records/assets/semantic-ui/js/invenio_rdm_records/src/deposit/errors/FormFeedback.js

+5-1
Original file line numberDiff line numberDiff line change
@@ -201,9 +201,13 @@ class DisconnectedFormFeedback extends Component {
201201
...severityChecks,
202202
});
203203

204+
const noSeverityChecksWithErrors = Object.values(severityChecks).every(
205+
(severityObject) => severityObject.severity !== "error"
206+
);
207+
204208
// Determine final feedback without reassigning
205209
const feedback =
206-
errorSections && _isEmpty(flattenedErrors) && !_isEmpty(severityChecks)
210+
errorSections && _isEmpty(flattenedErrors) && noSeverityChecksWithErrors
207211
? "suggestive"
208212
: initialFeedback;
209213

0 commit comments

Comments
 (0)