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

[feat] stamp 화면 API 연동, 한경대 -> 한국교통대 migration #273

Merged
merged 13 commits into from
Oct 13, 2024

Conversation

easyhooon
Copy link
Collaborator

@easyhooon easyhooon commented Oct 12, 2024

feat)

  • stamp 화면 API 연동
  • custom error code(9000, 9001...) 를 처리하기 위해 QRScanViewModel 만의 handleException 을 추가했습니다.

style)

  • 한경대 -> 한국 교통대로 migration (카메라 중심 좌표, 폴리곤, 하드코딩 API 호출("한경대" -> "한국교통대")

help)

  • QRScan 인식은 정상적으로 이뤄지는데, QRScan 을 위한 카메라 화면 내에 조준선을 구현하려는데 쉽지않네요, (아 Box 로 하면 될지도?) 추가 구현이 필요할 것 같습니다. -> 구현 완료
  • 또한 QR 이 없는 다른 곳을 QRScan 카메라로 비추게 될 경우 노란색 원형 점이 생기는 이슈가 있는데(QR 인식을 위한 어떤 무언가인듯) 이거 없애고 싶은데 키워드를 모르겠네요 구글링 또는 질문 올려보겠습니다.

테스트 후 문제 사항 있으면 제보 부탁드립니다.

easyhooon and others added 9 commits October 6, 2024 02:51
kotlin 2.0.20 버전 과 kotlinx-serialization 1.7.3 버전으로 맞춤, agp 버전은 8.2.2 그대로

agp 버전을 올리면 이전의 문제가 발생함
ErrorResponse 내에 에러 코드에 따라 분기 처리
조준선은 어떻게 구현해야할지 연구중... xml 이었으면 쉽게 구현할 것 같은데
@easyhooon easyhooon added enhancement New feature or request help wanted Extra attention is needed design tasks releated to design labels Oct 12, 2024
@easyhooon easyhooon self-assigned this Oct 12, 2024
Copy link
Collaborator

@wjdtkdgns777 wjdtkdgns777 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!! 오늘은 제가 일정이 있어서 가는길에 한번 봤는데 잘 작성하신거같습니다!!
오늘밤 내일아침쯤에 한번 더 자세히 살펴보고 문제없으면 approve 하겠습니다!!

Copy link
Collaborator

@wjdtkdgns777 wjdtkdgns777 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!! 캘린더 라이브러리가 업데이트한 컴포즈 버전과 호환이 되지 않아 홈화면으로 갈시 앱이 죽는 문제를 수정해줬습니다. 업데이트된게 많다보니 살짝 해맸었네요 ㅋㅋ

@easyhooon
Copy link
Collaborator Author

수고하셨습니다!! 캘린더 라이브러리가 업데이트한 컴포즈 버전과 호환이 되지 않아 홈화면으로 갈시 앱이 죽는 문제를 수정해줬습니다. 업데이트된게 많다보니 살짝 해맸었네요 ㅋㅋ

홈 화면에서 크래시가 발생했었군여, 제가 체크했어야 했는데 감사합니당

@easyhooon easyhooon merged commit e6de136 into develop Oct 13, 2024
1 check passed
@easyhooon easyhooon deleted the feature/stamp-api branch October 13, 2024 04:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design tasks releated to design enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants