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

Feature/#305 스터디페이지 권한 수정 #307

Merged
merged 7 commits into from
Sep 27, 2024

Conversation

llddang
Copy link
Collaborator

@llddang llddang commented Sep 22, 2024

관련 이슈

작업 요약

  • 스터디페이지에서 스터디 수정/삭제/종료 버튼이 스터디장에게만 보이도록 수정

리뷰 요구 사항

  • 리뷰 예상 시간 : 1분

@llddang llddang added 🐞 버그 예기치 못한 동작 발생 🎨 디자인 CSS 등 사용자 UI 디자인 변경 labels Sep 22, 2024
@llddang llddang self-assigned this Sep 22, 2024
Copy link
Collaborator

@yeonddori yeonddori left a comment

Choose a reason for hiding this comment

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

shaun

+++
다음 QA 때는 커리큘럼 편집 모드나 스터디원 관리 버튼도 안 보이게 해야겠네요~!
스크린샷 2024-09-26 154953

@llddang
Copy link
Collaborator Author

llddang commented Sep 26, 2024

+++ 다음 QA 때는 커리큘럼 편집 모드나 스터디원 관리 버튼도 안 보이게 해야겠네요~! !

옹 그렇네요 이것도 반영해놓을 게요!

Copy link
Collaborator

@jasper200207 jasper200207 left a comment

Choose a reason for hiding this comment

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

👍
코멘트는 큰 문제는 없을 듯 해서 Approve 하겠습니다

deleteModalOpen={setIsDeleteModalOpen}
/>
<Grid gap="4" templateColumns={{ base: '', xl: '2fr 1fr' }} w="100%">
{user?.memberId === studyData?.studyLeaderId && (
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분 user가 null 이고 studyData도 null 이라면 True로 판단될 수 있을 듯 합니다.
studyData && user?.memberId === studyData.studyLeaderId && 로 고치는 것도 좋아보입니다.

Copy link
Collaborator

@mun-kyeong mun-kyeong left a comment

Choose a reason for hiding this comment

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

좋습니다~! 저도 비회원일때로 확인되더라구요 수고하셨습니다

…into Feature/#305-스터디페이지_권한_작업
@llddang llddang merged commit 5a00b4a into develop Sep 27, 2024
1 check passed
@llddang llddang deleted the Feature/#305-스터디페이지_권한_작업 branch September 27, 2024 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 디자인 CSS 등 사용자 UI 디자인 변경 🐞 버그 예기치 못한 동작 발생
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 권한이 없는 사용자에게 스터디 관련 수정 버튼이 뜨는 않도록 수정
4 participants