-
Notifications
You must be signed in to change notification settings - Fork 43
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
425 metadata tab edit page #2338
Conversation
b03df60
to
fa994b5
Compare
2aec29b
to
b9ef490
Compare
9d7ad89
to
384856f
Compare
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.
This could be done in a subsequent PR (as part of the same story), but the validation needs work—there should be inline validation messages, and the message at the top should be more readable, and should be in an error summary, with links to the appropriate field.
I thought we were transitioning by content type—so you would only see the design system tabs for help pages and answers, but would still see the old bootstrap UI for other content types—has this changed?
Is there any test coverage for the validation messages?
13c9bca
to
7a86066
Compare
Add success message on update
3130caa
to
8fb9944
Compare
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.
A few comments related to the test file structure.
893fa22
to
f753276
Compare
f753276
to
f44bbf0
Compare
Trello
Move Metadata tab to new design system