-
Notifications
You must be signed in to change notification settings - Fork 2
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] 타임블록 파일 업로드 모달 구현 #452
[Feat] 타임블록 파일 업로드 모달 구현 #452
Conversation
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.
고생하셨습니다~~!! 모달은 이제 뚝딱하면 만들어내는 장인이네요!
@@ -25,7 +28,7 @@ const UploadedDocuments = ({ isEditable }: UploadedDocumentsProps) => { | |||
</Text> | |||
</Flex> | |||
{isEditable && ( | |||
<Button variant="outline" size="medium" style={{ width: '100%' }}> | |||
<Button variant="outline" size="medium" style={{ width: '100%' }} onClick={() => openModal('timeblock-file')}> |
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.
친히 연결까지 너무 감사합니다!!
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.
LGTM 모달 두개를 동시에!! 수고하셨습니다
해당 이슈 번호
closed #448
체크리스트
💎 PR Point
기존에 타임블록 생성할 때 있던 파일 업로드 모달 세 개를 빼서 새로운 Funnel 구조를 만들었습니다.
BrowseFileModal (기존에 업로드된 파일 중에서 파일을 선택하는 모달)은 재사용이 가능해보이는데 ,, 리소스 부족으로 shared에 파일을 새로 팠습니다. 리팩할 때 하나로 통합할게요 !
useTimeBlockUploadFlow.tsx
퍼널 구조는 다음과 같습니다.
이미 업로드된 파일들이 있는 경우에는
BrowseFileModal
을 띄워주며, 그렇지 않은 경우NewFileImportModal
을 띄웁니다.파일을 선택한 이후
SelectedFileModal
을 통해 선택된 파일들을 확인합니다SelectedFileModal
의완료
버튼에 대한 핸들러는handleComplete
로 넘겨주면 됩니다 !📌스크린샷 (선택)