✨ Feat(#155): 대시보드 공유 기능

un0211 commented 3 months ago

연관된 이슈

close #155

작업 내용

[x] 공유용 계정 생성 및 계정의 토큰 저장
[x] API를 이용해 공유시 공유용 계정을 추가
[x] 공유용 계정으로 공유된 대시보드에 접근 (axios get 요청 변경)
[x] 공유 취소 시 공유용 계정 멤버 삭제
[x] 공유용 계정 멤버 삭제 시 공유 토글 off
[x] 공유 페이지 로그인/비로그인 상태에 따른 디자인
- 로그인시 본인의 사이드바, 헤더 그대로 (헤더에는 열람중인 대시보드 정보)
- 비로그인시 사이드 바 없고 헤더는 로그인

스크린샷

공유 안한 생성

https://github.com/Part3-Team15/taskify/assets/24778465/f0e5ed4c-1751-4c44-a73d-add0ac107fd2

공유 한 생성

https://github.com/Part3-Team15/taskify/assets/24778465/58df4113-2ef6-4581-bda8-6842aef2a588

공유취소

https://github.com/Part3-Team15/taskify/assets/24778465/4b6def23-f8eb-47d8-ab6f-04400588a354

공유로 변경

https://github.com/Part3-Team15/taskify/assets/24778465/728e4094-6184-4e60-b5e2-fc85d06c3268

공유한 대시보드 권한없는 유저 확인

https://github.com/Part3-Team15/taskify/assets/24778465/a690df44-cdb1-4c73-a0c3-79fff734fb3c

코멘트

유저 정보 없으면 헤더는 안뜹니다! 괜찮을까요?

un0211 commented 3 months ago

이것도 완료되었으니 리뷰 부탁드려요~~

wjsdncl commented 3 months ago

잘되네요 고생하셨습니다! 🫡 추가적으로 공유계정용 헤더 추가랑 공유 계정은 상단 멤버표시에 안나오게 하는게 더 좋을것 같습니다

un0211 commented 3 months ago

안그래도 그게 문제였는데

공유계정으로 열람할 때 헤더가 필요할지
필요하다면 제목만 넣고 (원래 로그인 한 사람이면 자기정보는 넣을지)
사이드바(공유된 모든 대시보드가 사이드바에 뜰지도..?)

이 문제는 회의때 좀 더 얘기해봐야 할 것 같이요~

wjsdncl commented 3 months ago

사이드바 제거 혜원님이 하실건가요? 아니면 제가 이 pr에서 작업할까요?

un0211 commented 3 months ago

민재님이 하셔도 되는데 당장은 공유계정이라는 걸 알 방법이 없긴 합니다! 헤더는 제가 새로 만들어서 추가할게요. 검은색 이상할 것 같아서 ㅠ

wjsdncl commented 3 months ago

민재님이 하셔도 되는데 당장은 공유계정이라는 걸 알 방법이 없긴 합니다! 헤더는 제가 새로 만들어서 추가할게요. 검은색 이상할 것 같아서 ㅠ

레이아웃에 아래처럼 추가하면 헤더만 나올겁니다!

  if (currentPath === '/dashboard/[id]' && !user) {
    return (
      <div className='min-w-[375px]'>
        <Header />
        {children}
      </div>
    );
  }

un0211 commented 3 months ago

아~~ 로그인 된 유저는 원래 거 보여주기로 했었네요~~

un0211 commented 3 months ago

디자인 변경했습니다. 권한도 수정해야하는데 커질 것 같아서 다음에 할지, 묶어서 할 지 고민입니다. (일단 지금은 로그인 안되면 공유 페이지에서도 로그인 하라고 함 + 권한없는 대시보드 / 대시보드 수정에 접근 시 리다이렉션 해야함)

공유 대시보드