-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Accordion: add error labels in title & add FeedbackLabel #276
Conversation
32df2a7
to
0bb53a4
Compare
0bb53a4
to
8524ed7
Compare
b109e08
to
625a145
Compare
625a145
to
442b4f2
Compare
442b4f2
to
99a48c0
Compare
} | ||
} | ||
|
||
return { flattenedErrors, severityChecks }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Put a deprecation warning on flattenedErrors i.e. old error format
5140e32
to
7dd7e0a
Compare
// React-Invenio-Forms is free software; you can redistribute it and/or modify it | ||
// under the terms of the MIT License; see LICENSE file for more details. | ||
|
||
export function flattenAndCategorizeErrors(obj, prefix = "") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is prefix?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the prefix param is used to track the hierarchy of keys when recursively flattening nested objects
7dd7e0a
to
19486af
Compare
19486af
to
f78461c
Compare
❤️ Thank you for your contribution!
Description
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:
Frontend
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that: