Skip to content
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] 대시보드 뷰 1차 기능 QA #438

Merged
merged 5 commits into from
Jan 30, 2025

Conversation

Bowoon1216
Copy link
Contributor

@Bowoon1216 Bowoon1216 commented Jan 28, 2025

해당 이슈 번호

closed #434


체크리스트

  • 🔀 PR 제목의 형식을 잘 작성했나요? e.g. [feat] PR을 등록한다.
  • 💯 테스트는 잘 통과했나요?
  • 🏗️ 빌드는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요?
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 💻 git rebase를 사용했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

📌 내가 알게 된 부분


💎 PR Point

  • 인수인계 노트 조회 in 대시보드 뷰

  • 노트 클릭시 자세히 보기 뷰로

  • 파일 클릭 시 드라이드뷰로

  • 아직 못한 것: 블록 클릭시 사이드바 뜨도록하는것


📌스크린샷 (선택)

Copy link

🚀 Storybook 확인하기 🚀

@wuzoo
Copy link
Contributor

wuzoo commented Jan 29, 2025

고생하셨습니다 ! 아직 못한 기능 이라는 것은 해당 PR에서의 태스크를 다 마무리하지 못하신건가요, 아니면 다른 브랜치에서 진행하실 계획이신건가용 ?

Copy link
Member

@namdaeun namdaeun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

보우나 고생해써 !! ㅎㅎ LGTM

@@ -30,7 +33,11 @@ const FileSection = () => {
const allFileData = [...docDataList, ...folderDataList];

return (
<Flex css={[{ gap: '1.4rem', padding: '0 0 0.7rem', overflowX: 'scroll' }, scrollStyle, dashboradScrollStyle]}>
<Flex
css={[{ gap: '1.4rem', padding: '0 0 0.7rem', overflowX: 'scroll' }, scrollStyle, dashboradScrollStyle]}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

요기 dashboradScrollStyle과 scrollStyle이 어떻게 다른 걸까유 ?
하나로 지정해줘도 될 것 같아요

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아 스크롤 스타일은 글로벌스타일에서 가져온거구요
dashboardScrollStyle은 추가로 대시보드에서만 쓰이는 스크롤스타일 지정해줬습니다!

@Bowoon1216
Copy link
Contributor Author

고생하셨습니다 ! 아직 못한 기능 이라는 것은 해당 PR에서의 태스크를 다 마무리하지 못하신건가요, 아니면 다른 브랜치에서 진행하실 계획이신건가용 ?

원래 이 피알에서 하려고 했는데 좀 오래걸릴 것 같아서 다른 브랜치에서 다른 QA들이랑 같이 해야할 것 같습니다!

Copy link

🚀 Storybook 확인하기 🚀

@Bowoon1216 Bowoon1216 merged commit 9917b57 into develop Jan 30, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0차 QA 버그 해결
3 participants