Skip to content

🔀 :: (#54) Role Check 페이지를 퍼블리싱 했습니다. #55

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

Merged
merged 5 commits into from
Jun 24, 2024

Conversation

Chaejongin12
Copy link
Contributor

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

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

  • RoleCheckScreen 퍼블리싱

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

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

  • feature:login Module에 RoleCheckScreen 추가

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

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

  • 없음

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

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

  • Component화 할만한 Component가 많이 없어 단일 스크린으로 제작했습니다. 더 좋은 방법이 있다면 알려주세요.
  • 컨벤션에 맞지 않는 부분, 개행이 안되어 있는 부분 등 부족한 부분이 있다면 확인해주세요.

:: RoleCheckScreen 퍼블리싱
@Chaejongin12 Chaejongin12 self-assigned this Jun 24, 2024
@Chaejongin12 Chaejongin12 linked an issue Jun 24, 2024 that may be closed by this pull request
@minStone-dev
Copy link
Contributor

common 모듈의 로컬파일 몇개 올라왔네요?

:: gitinore 적용
:: strings.xml RoleCheckScreen의 Text 추가 및 stringResource처리
:: gitignore update
:: fix untracked file
@Chaejongin12
Copy link
Contributor Author

common 모듈의 로컬파일 몇개 올라왔네요?

수정했습니다

@Chaejongin12 Chaejongin12 merged commit ce9eafb into develop Jun 24, 2024
1 check passed
@Chaejongin12 Chaejongin12 deleted the feature/#54_publishing_role_check_screen branch June 24, 2024 05:51
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.

사용자의 Role Check Screen을 퍼블리싱합니다.
3 participants