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

[Fix] 1차 자체 QA 진행 #153

Closed
8 tasks done
EunsuSeo01 opened this issue Aug 6, 2024 · 0 comments · Fixed by #160
Closed
8 tasks done

[Fix] 1차 자체 QA 진행 #153

EunsuSeo01 opened this issue Aug 6, 2024 · 0 comments · Fixed by #160
Assignees
@EunsuSeo01
Labels
🐬 은수 은수 공주 작업 🔧 Chore Feat 이외에 코드 수정, 내부 파일 수정, 애매한 것들이나 잡일 🛠️ Fix 버그, 오류 해결

Comments

@EunsuSeo01
Copy link
Member

EunsuSeo01 commented Aug 6, 2024 •
edited
Loading

🛠 Issue

  • 앱잼 때 놓쳤던 부분 구현 및 버그를 수정합니다.
  • 회의 때 이야기 했던 부분의 컨벤션 통일과 코드 정리도 같이 진행할 예정입니다.

📝 To-do

코드 정리

  • 맡은 파일 StringLiterals 작성 완료
  • 컨벤션 통일 및 자잘한 코드 수정

놓쳤던 부분 & 버그 픽스

  • 토스트 뷰 띄우기 (검정, 하양)
  • 메뉴 가격 셀 하단 에러 메세지 없어지는 버그 픽스
  • 장소 검색 시 이미 제보된 식당일 때 분기 처리
  • 내 족보 바텀 시트에서 해당 식당이 이미 추가된 족보는 비활성화 처리

추가 요청 사항 구현

  • 제보할 때 이미지 1MB 이하로 압축하기
  • 디자인 문구 수정 사항 반영
@EunsuSeo01 EunsuSeo01 added the 🐬 은수 은수 공주 작업 label Aug 6, 2024
@EunsuSeo01 EunsuSeo01 self-assigned this Aug 6, 2024
@EunsuSeo01 EunsuSeo01 changed the title [Fix] 자체 QA 1차 [Fix] 1차 자체 QA 진행 (제보하기 플로우) Aug 6, 2024
@EunsuSeo01 EunsuSeo01 added 🛠️ Fix 버그, 오류 해결 🔧 Chore Feat 이외에 코드 수정, 내부 파일 수정, 애매한 것들이나 잡일 labels Aug 6, 2024
EunsuSeo01 added a commit that referenced this issue Aug 6, 2024
@EunsuSeo01
[Merge] #153 - branch 'refactor/#142' into fix/#153
b1dbc5f
EunsuSeo01 added a commit that referenced this issue Aug 6, 2024
@EunsuSeo01
[Chore] #153 - Type 명시
1a800bc
EunsuSeo01 added a commit that referenced this issue Aug 6, 2024
@EunsuSeo01
[Chore] #153 - 접근제어자 명시
a40cf18
EunsuSeo01 added a commit that referenced this issue Aug 6, 2024
@EunsuSeo01
[Refactor] #153 - init을 통해 주입하여 필수값임을 명시
84f8a94
EunsuSeo01 added a commit that referenced this issue Aug 6, 2024
@EunsuSeo01
[Chore] #153 - 코드 위치 변경
3060d22
EunsuSeo01 added a commit that referenced this issue Aug 6, 2024
@EunsuSeo01
[Chore] #153 - 불필요한 코드 삭제
7e8708e
EunsuSeo01 added a commit that referenced this issue Aug 6, 2024
@EunsuSeo01
[Chore] #153 - 불필요한 파라미터 삭제 및 중복 코드 함수로 분리
f04ef9d
EunsuSeo01 added a commit that referenced this issue Aug 6, 2024
@EunsuSeo01
[Chore] #153 - 불필요한 함수 호출 삭제 및 함수 위치 이동
042a525
EunsuSeo01 added a commit that referenced this issue Aug 6, 2024
@EunsuSeo01
[Chore] #153 - 맡은 파일 전체 StringLiterals 작성 완료
cff74b1
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Fix] #153 - 내 족보 추가 완료 검정 토스트뷰 띄우기
261e07a 
내 족보 바텀 시트 내려간 후 Notification을 통해 홈 화면에서 토스트가 띄워질 수 있도록 수정
보기 버튼 터치 영역 확장
토스트뷰 제거 애니메이션과 버튼 클릭 이벤트 동시에 가능하도록 코드 수정함
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Merge] #153 - pull develop
7990937
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Chore] #153 - handleNetworkResult 함수의 불필요한 파라미터 제거
faac03b
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Chore] #153 - handleNetworkResult 함수 적용
968a50b
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Fix] #153 - 식당 검색에 univ랑 location 데이터 연결
c53c952
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Feat] #153 - 이미지 압축을 위한 ImageCompressor 구현
5d9e14f
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Feat] #153 - 최대 압축해도 1MB 넘는 경우 예외 처리 로직 추가
38382af
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Merge] #153 - pull develop
c100f26
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Feat] #153 - 내 족보 추가 완료 White Toast 띄우기
9694eff
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Fix] #153 - 식당 선택 시 이미 해당 대학교에 등록된 식당일 경우 에러 처리
3ba844c
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Fix] #153 - 가격 에러 메세지 안 보이는 버그 픽스 및 에러 스타일 설정하는 코드 수정
9443d38
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Chore] #153 - 맡은 셀 파일들 StringLiterals 작성
bda9935
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Chore] #153 - 타입 명시
ac1292f
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Chore] #153 - MyZip을 MyZipList로 폴더명 변경 및 내 족보 바텀시트 썸네일 에셋 변경
7aa3dfa
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Fix] #153 - 새로운 족보 추가하기 클릭 영역 확대
b1aabf8 
이미지 버튼 클릭해도 작동하도록
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Fix] #153 - 해당 식당이 이미 추가된 족보일 경우 비활성화 처리하여 버그 수정
09b6051
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Fix] #153 - 제보 후 내 족보에 추가 시 다른 족보에도 추가 텍스트로 바로 변경
5d7cd2d
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Chore] #153 - NotificationName 변경
0badee0
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Chore] #153 - 문구 수정사항 반영
ee46b9d
EunsuSeo01 added a commit that referenced this issue Aug 9, 2024
@EunsuSeo01
[Chore] #153 - 문구 수정사항 반영
39c5a42
@EunsuSeo01 EunsuSeo01 changed the title [Fix] 1차 자체 QA 진행 (제보하기 플로우) [Fix] 1차 자체 QA 진행 Aug 9, 2024
@EunsuSeo01 EunsuSeo01 mentioned this issue Aug 9, 2024
Merged
3 tasks
@EunsuSeo01 EunsuSeo01 linked a pull request Aug 9, 2024 that will close this issue
[Fix] #153 - 1차 자체 QA 반영 #160
Merged
3 tasks
EunsuSeo01 added a commit that referenced this issue Aug 10, 2024
@EunsuSeo01
[Chore] #153 - Utils 폴더 생성 및 파일 이동
ba06cf6
EunsuSeo01 added a commit that referenced this issue Aug 10, 2024
@EunsuSeo01
[Merge] #153 - 1차 자체 QA 반영
abd9556
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@EunsuSeo01 EunsuSeo01

Labels
🐬 은수 은수 공주 작업 🔧 Chore Feat 이외에 코드 수정, 내부 파일 수정, 애매한 것들이나 잡일 🛠️ Fix 버그, 오류 해결
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Fix] #153 - 1차 자체 QA 반영 Team-Hankki/hankki-iOS
1 participant
@EunsuSeo01