-
Notifications
You must be signed in to change notification settings - Fork 137
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
feat: add customSvg
prop to FullPageError
#5976
base: main
Are you sure you want to change the base?
feat: add customSvg
prop to FullPageError
#5976
Conversation
DCO Assistant Lite bot All contributors have signed the DCO. |
✅ Deploy Preview for carbon-for-ibm-products ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
4cb6af3
to
ec34195
Compare
I have read the DCO document and I hereby sign the DCO. |
recheck |
ec34195
to
fb4a8a1
Compare
Signed-off-by: Stefan Cooper <[email protected]>
fb4a8a1
to
d090c41
Compare
waiting for issue triaging/design opinion as a first pass. |
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.
Looks good to me, like @devadula-nandan said lets see what design says
Closes #5975
This PR adds a new optional prop to the
FullPageError
component. It allows you to provide a custom JSX/svg instead of the generic svg for 'custom' full page errorsWhat did you change?
I added a new optional property that is used when
kind
iscustom
andcustomSvg
is defined. Otherwise, it will use the original behavior.How did you test and verify your work?
I added a test that uses the 403 svg on the 'custom' kind and verified that it was found.