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

워크스페이스 초대, protected route 수정 #380

Merged
merged 1 commit into from
Dec 3, 2024
Merged

Conversation

djk01281
Copy link
Collaborator

@djk01281 djk01281 commented Dec 3, 2024

🔖 연관된 이슈

📂 작업 내용

  • protected route으로 이동 시, user가 fetch되기 전에 validate해서 권한이 없는 것으로 처리되는 문제
  • /join 에서 워크 스페이스 ID를 잘못 가져오는 문제
  • 공유 버튼을 누르기만 해도 private일 경우 링크를 가져오는 문제 (copy 버튼 클릭 시 가져오게 수정)

@djk01281 djk01281 added 🐰🐣 FE 프론트엔드 관련 이슈/PR 추적 라벨 🐛 Bug 무언가 정상적으로 동작하지 않는 경우 labels Dec 3, 2024
@djk01281 djk01281 self-assigned this Dec 3, 2024
@djk01281 djk01281 marked this pull request as draft December 3, 2024 07:17
@djk01281 djk01281 marked this pull request as ready for review December 3, 2024 07:25
@djk01281 djk01281 requested a review from yewonJin December 3, 2024 07:26
Copy link
Collaborator

@yewonJin yewonJin left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!

@yewonJin yewonJin merged commit 231a0f7 into develop Dec 3, 2024
20 checks passed
@yewonJin yewonJin deleted the bug-fe-#379 branch December 3, 2024 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug 무언가 정상적으로 동작하지 않는 경우 🐰🐣 FE 프론트엔드 관련 이슈/PR 추적 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

protected route, 초대 관련 버그 수정
2 participants