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.

Sign up for GitHub

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

Jump to bottom

[ Feature ] : QR 체크 -> 암호 입력 변경 #319

Merged
merged 8 commits into from
Oct 27, 2023
Merged

[ Feature ] : QR 체크 -> 암호 입력 변경 #319

merged 8 commits into from
Oct 27, 2023

Conversation

TaeseongYun
Copy link
Contributor

Description
QR 체크 -> 암호 입력 변경

  • 라우터가 QR_CODE 에서 PASSWORD 로 변경되었습니다.
  • PASSWORD 라우터가 추가되었습니다.
  • Password content 가 Image 사용에서 Text 사용으로 변경되었습니다.
yapp_attendance.mp4

Check List

  • CI/CD 통과여부
  • Develop Mege 시 Squash and Merge 형태로 넣어주세요!
  • 1Approve 이상 Merge
  • Unit Test 작성
  • 연관된 이슈를 PR에 연결해주세요.

EvergreenTree97
EvergreenTree97 approved these changes Oct 26, 2023
View reviewed changes
Copy link
Collaborator

@EvergreenTree97 EvergreenTree97 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

presentation/src/main/java/com/yapp/presentation/ui/member/signup/password/Password.kt Outdated
@@ -190,9 +190,9 @@ internal fun PasswordTextField(
modifier = Modifier.fillMaxWidth(),
horizontalArrangement = Arrangement.Absolute.SpaceBetween,
) {
value.forEachIndexed { index, _ ->
value.forEach { value ->
Copy link
Collaborator

Choose a reason for hiding this comment

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

value라는 네이밍이 겹쳐서 password 같은 네이밍으로 변경해도 좋을 것 같네요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

해당 리뷰는 디자인 리셋으로 인해 기존코드 유지로 변경되었습니다 !

@TaeseongYun
Copy link
Contributor Author

변경 되어야 할 리스트

  • 디자인 ( 기존 디자인 유지 ) , 타이틀 변경
  • 암호 출석 완료 후처리 필요

@TaeseongYun
Copy link
Contributor Author

ISSUE-317.after.mp4

관련해서 작업 완료하였습니다.

Feature

  • MainViewModel 클래스 MarkAttendanceUseCase 추가
  • DateUtil parse 함수 추가

Refactor

  • Password 컴포즈 화면 스트링 리소스 변경 ( R.string.member_signup_password_title )
  • QR 코드 라우팅 변경

@jihee-dev
Copy link
Member

이거 회원 가입을 위한 진입이랑 출석체크를 위한 진입이 로직이 달라서 처리가 필요할 것 같아요

지금 회원 가입 하려고 들어왔을 때 암호로 되어 있고, 또 문구도 가입용 문구가 아니라 출석용 문구가 보이네요 ㅠㅠ

@TaeseongYun
Copy link
Contributor Author

ISSUE-317.after-2.mp4

@jihee-dev 님
확인해보니 회원가입 시 SignUP_NAME 으로 라우팅이 아닌 패스워드로 라우팅을 시키고 있어서 문제가 되었습니다.

추가로 버튼 크기가 기기마다 맞지 않은거 같은데 해당 이슈는 추가 개선사항으로 가져가겠습니다 ~

@toastmeister1 toastmeister1 merged commit 8fdf895 into develop Oct 27, 2023
@toastmeister1 toastmeister1 deleted the feature/ISSUE-317 branch October 27, 2023 15:05
@TaeseongYun TaeseongYun linked an issue Oct 29, 2023 that may be closed by this pull request
암호 출석 기능 구현 #317
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EvergreenTree97 EvergreenTree97 EvergreenTree97 approved these changes

Assignees

@TaeseongYun TaeseongYun

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

암호 출석 기능 구현
4 participants
@TaeseongYun @jihee-dev @EvergreenTree97 @toastmeister1