feat: created new troubleshoot component#352
Conversation
🦋 Changeset detectedLatest commit: 30c44f5 The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Ishaan28malik
force-pushed
the
feature/TroubleShootDialog
branch
from
520556c to
bacda42
Compare
packages/cysync-core/src/dialogs/TroubleShoot/Dialogs/CloseConfirmation.tsx
Outdated
Show resolved
Hide resolved
packages/cysync-core/src/dialogs/TroubleShoot/context/index.tsx
Outdated
Show resolved
Hide resolved
packages/cysync-core/src/dialogs/TroubleShoot/context/index.tsx
Outdated
Show resolved
Hide resolved
packages/cysync-core/src/dialogs/TroubleShoot/context/index.tsx
Outdated
Show resolved
Hide resolved
Ishaan28malik
force-pushed
the
feature/TroubleShootDialog
branch
from
35f4ac4 to
0fcad44
Compare
Ishaan28malik
force-pushed
the
feature/TroubleShootDialog
branch
from
ca04952 to
428c01c
Compare
irshadCypherock
left a comment
irshadCypherock
left a comment
There was a problem hiding this comment.
- Alignment issues
packages/cysync-core/src/pages/MainApp/Settings/Tabs/AppSettings.tsx
Outdated
Show resolved
Hide resolved
| onPrevious: React.MouseEventHandler<HTMLButtonElement>; | ||
| isFirstDialog?: boolean; | ||
| } | ||
| export const UsbTroubleShootDialogBox: FC<UsbTroubleShootDialogBoxProps> = ({ |
There was a problem hiding this comment.
This component is almost a duplicate of GuidedFlowDialogBox. Prefer reusing the same component to avoid duplicate code.
There was a problem hiding this comment.
created a reusable component as CustomFloeDialogBox
| return ( | ||
| <WaitingDiv | ||
| key={`${type}-${index + 1}`} | ||
| text={messageBox[key]} | ||
| textColor={textColor} | ||
| type={type} | ||
| /> | ||
| ); |
There was a problem hiding this comment.
This assumes that all the messageBoxList element should be displayed with a loader, which might not be true.
There was a problem hiding this comment.
This is Didn't understood @irshadCypherock
2 participants
Added new Dialog for
USBTroubleShoot