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

디자인 수정 + 매장 방문자 리스트 확인 화면 추가 #30

Merged
merged 9 commits into from Oct 4, 2021

Conversation

Jsueeee
Copy link
Member

@Jsueeee Jsueeee commented Oct 2, 2021

  1. 스피커 앱과 비슷하게 디자인을 전면 수정했습니다. (문구 수정 진행중)

  2. 매장에서 방문자의 리스트를 확인할 수 있도록 방문기록 화면을 추가했습니다.

  3. 기존 메인액티비티에 있던 csv 파일 추출 버튼은 화면 구성 상 매장 방문 기록 조회 화면에 위치하는 것이 낫겠다는 생각이 들어 위치를 바꿨습니다. (함수들 위치도 VisitorListActivity, ViewModel로 변경) -> 원래 위치가 더 보기 좋을 것 같다는 의견도 좋습니다! 코멘트 부탁드려요.

  4. 방문자 리스트 확인을 위한 리사이클러뷰를 넣었습니다.

  5. 저번에 @sujin9 수진님이 레포지토리 사용을 권유해주셨는데, 리사이클러뷰를 넣는 과정에서 같이 넣어보았습니다.
    수진님이 만드신 csvFile 추출 쪽의 getAll은 List 형으로 일단 놔뒀습니다. 라이브데이터로 바꾸실 의향 있으면 한 번 봐주세요!
    (리사이클러뷰에서 데이터가져오는 부분은 라이브데이터형으로 만들어놨습니다.)

커밋을 나름 세분화하려고 해서, 많긴 하지만 커밋별로 봐주시면 감사하겠습니다 :)

@Jsueeee Jsueeee requested review from eunzinn and a team and removed request for eunzinn October 2, 2021 09:48
Copy link
Contributor

@YoungSeokHong YoungSeokHong left a comment

Choose a reason for hiding this comment

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

리사이클러 뷰에, UI전면 개편에 엄청 많이 바꼈네요. 너무 고생 많으셨습니다ㅠㅠ
저는 리사이클러 뷰 하나 만드는 것도 일이던데ㅋㅋㅋ 정말 말씀하신대로 뚝딱 만드셨네요. 감사합니다!
추출 버튼 옮기신건 아까 발표중에 보여주신 것처럼 되있는걸 말씀하시는거군요. 저는 지금 하신 것처럼 추출버튼을 집어 넣은게 더 마음에 듭니다!
정말 고생 많으셨습니다! 머지 해주시면 다운 받아본 뒤에 열심히 테스트 해보겠습니다!

@sujin9
Copy link
Member

sujin9 commented Oct 2, 2021

너무 수고 많으셨어요!!
그리고 @YoungSeokHong 님 코멘트 보고
혹시 도움이 될 것 같아서 디스커션 #34 남겨보았어요!

@eunzinn
Copy link
Contributor

eunzinn commented Oct 2, 2021

와 너무 많은 양인데 정말정말정말 수고 많으셨습니다..ㅠㅠ

@dlwlstks96
Copy link
Contributor

확인했습니다! 바쁘신 와중에 고생 많으셨습니다!!
화면과 기능에 맞게 코드도 더욱 깔끔해진 것 같아 좋습니다:) 👍👍

@Jsueeee Jsueeee merged commit 84a2aef into euphony-io:master Oct 4, 2021
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.

None yet

5 participants