🛠 Fix(#168): 권한에 따른 페이지 접근 제어

[un0211]

연관된 이슈

• 168

• 82

작업 내용

• 이전에 사용하던 리다이렉트 훅 리팩토링
  • 최상위에서 쓰일 수 있도록 변경 (조건문 밖으로)
• 랜딩 페이지
  • 모달 없이 리다이렉트
• 404
  • 3초 뒤 로그인 or 나의 대시보드로 이동
• 대시보드 페이지
  • 공유 대시보드 접근 가능
  • 공유 대시보드가 아니면 권한 없다고 하고 리다이렉트
  • 로그인 상태 -> 나의 대시보드
  • 비로그인 상태 -> 로그인
• 대시보드 관리 페이지
  • 대시보드 생성자가 아니면 권한 없다고 리다이렉트
  • 로그인 상태 -> 나의 대시보드
  • 비로그인 상태 -> 로그인
• 공유 대시보드 수정
  • 레이아웃 수정
  • 카드 확인 이외 수정 모두 불가능
• 대시보드 수정 페이지 에러 메시지 출력 예쁘게 (어차피 리다이렉트 되면 볼 일 없음

스크린샷

공유 대시보드

https://github.com/Part3-Team15/taskify/assets/24778465/169eb584-d384-4abd-b33d-a6241cf7d332

코멘트 및 논의 사항

axios나 useQuery 사용해서 에러코드 받아서 에러처리 하는 게 깔끔한 것 같은데, 당장은 방법을 몰라서 에러 핸들링 방식으로 처리했습니다! 일단 위에 적은 대로 동작하고, 변경 원하시는 부분 있으면 코멘트 달아주세요! 조금 더 깔끔한 리다이렉션 가능하면 추후 수정해보겠습니다.

[wjsdncl]

팀 미팅 때 말씀하셨던 거처럼 로그인 상태에서 랜딩페이지 이동했을 때 리다이렉트는 모달 안 띄우는 게 더 좋을 거 같습니다. 보통 사이트 진입을 랜딩으로 하기 때문에 계속 모달 나오면 사용자 경험에 안 좋을 거 같네요

코드잇 사이트처럼 404페이지에서는 몇초 뒤에 랜딩으로 이동하는 거는 어떨까요?

공유 대시보드 사이드바 없으니까, PC에서 잘리네요

공유 대시보드의 임시 계정은 말씀하셨던 거처럼 읽기만 가능한 게 좋을 거 같네요

나머지는 잘 작동하는 거 같습니다. 고생하셨습니다! 🫡👍

[un0211]

팀 미팅 때 말씀하셨던 거처럼 로그인 상태에서 랜딩페이지 이동했을 때 리다이렉트는 모달 안 띄우는 게 더 좋을 거 같습니다. 보통 사이트 진입을 랜딩으로 하기 때문에 계속 모달 나오면 사용자 경험에 안 좋을 거 같네요

코드잇 사이트처럼 404페이지에서는 몇초 뒤에 랜딩으로 이동하는 거는 어떨까요?

추가할게요!!

공유 대시보드 사이드바 없으니까, PC에서 잘리네요

제가 태블릿, 모바일로만 봐서 몰랐네요! 이건 대시보드 CSS에서 사이드바 뺀 크기만큼 가지라고 해서 그런건데, 수정해보겠습니다.

공유 대시보드의 임시 계정은 말씀하셨던 거처럼 읽기만 가능한 게 좋을 거 같네요

이건 이 PR 말고 다른 PR에서 할 일이라고 생각했는데, (리다이렉트만 하는 PR이라고 생각해서) 접근제어니 여기서 해도 괜찮겠네요! 수정할게요~~

[un0211]

404에서 3초 뒤 다른 페이지로 이동하도록 설정했습니다!!

[un0211]

공유 대시보드 권한 다 막은 줄 알았더니 로그인 안한 사람만 막고 로그인 한 사람은 안막았네요..

[un0211]

https://github.com/Part3-Team15/taskify/assets/24778465/2e970a32-a662-444d-9986-678d08109548

대시보드 멤버 아닌 경우 이정도로 충분할까요? 아니면 로그인 안한 상태처럼 버튼 자체가 비활성화 되는게 나을까요?!!

[wayandway]

2024-07-06.12.34.49.mov 대시보드 멤버 아닌 경우 이정도로 충분할까요? 아니면 로그인 안한 상태처럼 버튼 자체가 비활성화 되는게 나을까요?!!

버튼 자체가 비활성화 되는 것이 좋을 것 같습니다!

[wjsdncl]

2024-07-06.12.34.49.mov 대시보드 멤버 아닌 경우 이정도로 충분할까요? 아니면 로그인 안한 상태처럼 버튼 자체가 비활성화 되는게 나을까요?!!

네 수정 모달을 아에 못띄우게 버튼을 막아버리는게 좋을거 같습니다

[un0211]

멤버가 아닌 경우 버튼 비활성화했습니다!! 추가로 리다이렉트 하는 부분 리팩토링하고 싶긴 한데, 이 PR 닫고 새 PR에서 할까 합니다!! 다른분들 테스트 부탁드려요

[un0211]

b179824 @wayandway 머지했는데 - 인 부분 중 일부러 지운 게 아닌 것 있으면 알려주세요

[wayandway]

b179824 @wayandway 머지했는데 - 인 부분 중 일부러 지운 게 아닌 것 있으면 알려주세요

넵 문제없는 거 확인했습니다!