Skip to content

🔀 :: (#120) Role Of Student Not Accept SignIn#121

Open
yeongun130 wants to merge 2 commits intodevelopfrom
feature/120_student_role_not_sign_in_accept
Open

🔀 :: (#120) Role Of Student Not Accept SignIn#121
yeongun130 wants to merge 2 commits intodevelopfrom
feature/120_student_role_not_sign_in_accept

Conversation

@yeongun130
Copy link
Contributor

🔍 이 PR을 통해 해결하려는 문제가 무엇인가요?

어떤 기능을 구현할건지, PR이 열리게 된 계기와 목적을 Reviewer들이 이해할 수 있게 적어주세요.

  • 학생 권한 일 경우 선생님 회원 가입 수락 버튼이 띄워지지 않게 수정했습니다.

✨ 이 PR에서 핵심적으로 변경된 사항은 무엇인가요?

문제를 해결하면서 주요하게 변경된 사항을 적어주세요.

🔖 핵심 변경 사항 이외에 추가적으로 변경된 부분이 있나요?

없으면 "없음"이라고 기재해주세요.

  • 없음

🧐 Review 시 이런 부분을 신경써서 봐주시면 좋겠어요

개발 과정에서 다른 분의 의견이 궁금하거나 크로스 체크가 필요하다고 느껴진 코드가 있다면 남겨주세요.

@yeongun130 yeongun130 self-assigned this Jul 15, 2024
@yeongun130 yeongun130 linked an issue Jul 15, 2024 that may be closed by this pull request
Comment on lines 109 to 112
JoinWaitingButton(
modifier = modifier.padding(bottom = 96.dp, end = 8.dp),
onClick = { openDialog = true }
)
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분을 플로팅버튼으로 처리하는게 어떨까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

786903b 수정했습니다

Copy link
Contributor

@minStone-dev minStone-dev left a comment

Choose a reason for hiding this comment

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

xml코드를 수정한 이유가 있으실까요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

학생 권한일 경우 회원 가입 수락 불가능 처리

3 participants