-
Notifications
You must be signed in to change notification settings - Fork 6
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(react): add providers & custom hooks #13
feat(react): add providers & custom hooks #13
Conversation
generates a theme that can be passed to the ThemeProvider
locale, | ||
providerCustomization.name, | ||
name: configData.name ?? 'carbon.super', |
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.
Let's move to a seperate constant?
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.
Wil address this in the next PR.
screen, | ||
providerCustomization.type, | ||
); | ||
type: configData.type ?? BrandingPreferenceTypes.Org, |
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.
Let's check and nullable checks if required.
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.
Wil address this in the next PR.
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 config cannot be nullable. Application cannot reach this far, if this is nullable. I think it is expected to break, in case this becomes nullable.
@@ -46,12 +47,17 @@ | |||
"sass": "^1.75.0", | |||
"stylelint": "15.1.0", | |||
"tslib": "^2.6.2", | |||
"typescript": "^5.4.5" | |||
"typescript": "5.1.6" |
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.
Is this intentional?
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.
caret ^
symbol will update to all future minor/patch versions, without incrementing the major version. (npm docs)
since current eslint version supports only upto typescript 5.2, need to avoid this.
Purpose
Related Issues
Related PRs
Checklist
Security checks