-
Notifications
You must be signed in to change notification settings - Fork 0
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
Maryia/TS-Package-7/add types for SmartCharts components #46
Maryia/TS-Package-7/add types for SmartCharts components #46
Conversation
export const StudyLegend = React.lazy(load('StudyLegend')); | ||
export const Views = React.lazy(load('Views')); | ||
export const ToolbarWidget = React.lazy(load('ToolbarWidget')); | ||
export const FastMarker = React.lazy(load('FastMarker')); |
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.
removed export const RawMarker = React.lazy(load('RawMarker'));
since it doesn't exist in flutter-chart branch of SmartCharts and not used here anymore.
portalNodeId?: string; | ||
}; | ||
|
||
type TSmartChartProps = React.PropsWithChildren<{ |
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.
types from the removed SmartChartSwitcher https://github.com/kate-deriv/deriv-app/pull/45/files#diff-04bb2774644da792dc593a913b0892dcd4d21c260415ddda81814bd041f85bb1
type TNotification = { | ||
category: string; | ||
text: string; | ||
type?: 'info' | 'warning' | 'success' | 'error' | 'warning'; |
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.
extra 'warning'
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.
I will merge this PR into the parent branch and will remove extra 'warning'
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.
thank you @kate-deriv :)
b814d0c
into
kate-deriv:kate/DTRA-420/ts_migration_7
Changes: