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] 2차 스프린트 기능 QA 반영 #289

Open
2 tasks done
EunsuSeo01 opened this issue Nov 10, 2024 · 0 comments · May be fixed by #294
Open
2 tasks done

[Fix] 2차 스프린트 기능 QA 반영 #289

EunsuSeo01 opened this issue Nov 10, 2024 · 0 comments · May be fixed by #294
Assignees
@EunsuSeo01
Labels
🐬 은수 은수 공주 작업 🛠️ Fix 버그, 오류 해결

Comments

@EunsuSeo01
Copy link
Member

EunsuSeo01 commented Nov 10, 2024 •
edited
Loading

🛠 Issue

  • 1108 2차 스프린트 기능 QA를 반영합니다.
  • 미완성이었던 메뉴 추가를 완벽히 구현합니다.

📝 To-do

  • 메뉴 추가 쪽 완성
  • 기능 QA 모두 반영 완료 - 이따 적겠음
@EunsuSeo01 EunsuSeo01 added 🛠️ Fix 버그, 오류 해결 🐬 은수 은수 공주 작업 labels Nov 10, 2024
@EunsuSeo01 EunsuSeo01 self-assigned this Nov 10, 2024
EunsuSeo01 added a commit that referenced this issue Nov 13, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 셀 포커스 시 border 컬러 변경
e2c00b9
EunsuSeo01 added a commit that referenced this issue Nov 13, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 셀 1개일 때 하단 버튼 텍스트 변경
111775a
EunsuSeo01 added a commit that referenced this issue Nov 13, 2024
@EunsuSeo01
[Feat] #289 - 메뉴 셀 1개일 때 x 버튼 숨김 처리 로직 추가
3aebbff 
제보하기의 메뉴 셀에도 동일하게 적용 완료함
EunsuSeo01 added a commit that referenced this issue Nov 13, 2024
@EunsuSeo01
[Fix] #289 - 가격 텍스트 필드 normal, focused, error 스타일 활성화 로직 수정
b44f6b5
EunsuSeo01 added a commit that referenced this issue Nov 14, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 추가 시 하단 버튼 활성화 타이밍 수정
8b0c33a
EunsuSeo01 added a commit that referenced this issue Nov 14, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 추가 및 수정 Req Body 구조체 재설정
3e7c773 
MenuData에서 MenuRequestDTO로 구조를 변경하는 toMenuRequestDTO 함수도 추가함
EunsuSeo01 added a commit that referenced this issue Nov 14, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 추가 url 수정
586acda
EunsuSeo01 added a commit that referenced this issue Nov 14, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 추가 완료 화면 구현 및 완료 화면 init 파라미터 추가
caa3d28
EunsuSeo01 added a commit that referenced this issue Nov 14, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 추가 후 식당 상세로 돌아왔을 때 높이 알맞게 늘어나도록 수정
c11a3a4 
removeConstraints 후 다시 makeConstraints 함으로써 높이 안 늘어나던 이슈 해결
EunsuSeo01 added a commit that referenced this issue Nov 14, 2024
@EunsuSeo01
[Fix] #289 - 완료 화면 네비바 없애기
7969db9
EunsuSeo01 added a commit that referenced this issue Nov 14, 2024
@EunsuSeo01
[Fix] #289 - viewWillAppear에서 메뉴 셀 초기화
774b67e 
다른 메뉴도 추가하기 클릭했을 때를 대비
EunsuSeo01 added a commit that referenced this issue Nov 14, 2024
@EunsuSeo01
[Fix] #289 - doThisAgainButton 클릭 시에도 postNotification 호출되도록 구조 변경
b54332b 
doThisAgainButton 클릭 후 네비바의 뒤로가기 버튼 클릭 시에 식당 상세 화면이 나오는데
이때에도 업데이트 된 최신 메뉴 리스트를 보여주기 위해서
EunsuSeo01 added a commit that referenced this issue Nov 14, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 불러오기 API 연동 및 연결
197caef 
API 구현 전이라 임의로 식당 상세 보기의 API에서 메뉴 리스트만 가져와서 썼던 걸 새 API로 변경
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - Response DTO 수정
1f8f19f 
서버가 dto 구조 하나로 통일해줘서 반영함
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 편집 화면에서 마지막 메뉴 삭제 시 Alert 문구 변경
d6378dc
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 마지막 메뉴 삭제 시 Alert 문구 및 로직 변경
d4e7184 
편집에서 바로 삭제 시 로직 변경
수정 중에 가격 넘어서 삭제할 때에도 적용함
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Chore] #289 - StringLiterals 작성
4be8f2b
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - VM의 modifiedMenuData 값을 선택된 메뉴 데이터로 초기화
e1715b6
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 기존 적용 버튼을 수정 완료 버튼으로 변경
f977c7b 
문구를 포함하여 버튼 로직 변경
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 하단 수정 완료 버튼 초기 스타일 활성화로 변경
5f437bb
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 가격 유효성 조건 추가
4263a3d 
가격이 0보다 커야 함
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 에러 스타일은 가격 텍스트필드일 때만 적용되도록 수정
624a152
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Chore] #289 - 변수명 변경
34e3ec4 
헷갈림
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 그냥 텍스트필드 클릭 시에도 수정 가능하도록 변경
aa3d659 
원래는 연필 버튼 클릭 시에만 수정 가능하게 막아뒀었는데 이를 없앰
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 가격 에러 스타일 변경
eaab9a0
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 가격 에러일 때만 deleteMenuAccessoryView 뜨도록 재수정
f14d99f
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
Merge branch 'develop' into fix/#289
f295b3a
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 텍스트필드 클릭 시 커서 맨 오른쪽으로
0acb686 
연필 버튼 클릭 시와 동일한 로직으로 동작하도록 탭 제스처 추가함
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 하단 수정 완료 클릭 시에도 모달 뜨도록 수정
4a4b5e7
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 하단 수정 완료 버튼 활성화 조건 변경
be56afe 
원래 값과 메뉴 이름 또는 가격이 다를 때에만 활성화 되도록
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 키보드 위의 수정 완료 버튼도 활성화 조건 변경
0913b53
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Chore] #289 - 마지막 메뉴 삭제 시 Alert 문구 변경
d89f17e
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 추가 화면 상단 네비바 설정
40a2bfb
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 메뉴 편집 바텀시트에서 네비바 제거
d5fe41b
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 기존 메뉴 값 입력 버튼 클릭 시 수정 완료 비활성화
3ba7a77
EunsuSeo01 added a commit that referenced this issue Nov 15, 2024
@EunsuSeo01
[Fix] #289 - 키보드 위 수정 완료 버튼 기능 비활성화 로직 추가
d76cbf3
EunsuSeo01 added a commit that referenced this issue Nov 16, 2024
@EunsuSeo01
[Chore] #289 - 안 쓰는 폴더 삭제
0dc5708
EunsuSeo01 added a commit that referenced this issue Nov 16, 2024
@EunsuSeo01
[Chore] #289 - final 키워드 빠진 부분 추가
41d67d4
EunsuSeo01 added a commit that referenced this issue Nov 16, 2024
@EunsuSeo01
[Refactor] #289 - storeId 위치 VC에서 VM으로
ce56dc3 
데이터는 VM이 관리
EunsuSeo01 added a commit that referenced this issue Nov 16, 2024
@EunsuSeo01
[Chore] #289 - 파라미터명과 함수명 변경
2faca07
EunsuSeo01 added a commit that referenced this issue Nov 16, 2024
@EunsuSeo01
[Refactor] #289 - 프로토콜 분리
e9b57d2 
하나의 프로토콜에 있던 함수 2개를 각각의 프로토콜로 분리했다
AddVC에서는 원래 프로토콜의 함수 중 1개의 함수만 사용했기 때문이다
EunsuSeo01 added a commit that referenced this issue Nov 17, 2024
@EunsuSeo01
[Fix] #289 - return 키 클릭 시 키보드 내리기
f6b303f
EunsuSeo01 added a commit that referenced this issue Nov 17, 2024
@EunsuSeo01
[Feat] #289 - 수정 시 메뉴 이름 최대 30자 제한 걸기
04eeb3c
EunsuSeo01 added a commit that referenced this issue Nov 17, 2024
@EunsuSeo01
[Feat] #289 - 편집 화면에서 메뉴 이름 길 때 잘리도록 레이아웃 설정
4c4506c
EunsuSeo01 added a commit that referenced this issue Nov 17, 2024
@EunsuSeo01
[Feat] #289 - 수정 화면에서 메뉴 이름 30자일 때에도 전부 보이도록 레이아웃 설정
85f1b48
EunsuSeo01 added a commit that referenced this issue Nov 17, 2024
@EunsuSeo01
[Refactor] #289 - 텍스트필드 활성화 로직 개선
af62f96 
로직을 간소화 하고 알아보기 쉽게 동작마다 함수로 분
EunsuSeo01 added a commit that referenced this issue Nov 17, 2024
@EunsuSeo01
[Refactor] #289 - 메뉴 추가 셀 스타일 변경 함수 개선
ebdcc71
@EunsuSeo01 EunsuSeo01 linked a pull request Nov 17, 2024 that will close this issue
[Fix] #289 - 2차 스프린트 기능 QA 반영 #294
Open
3 tasks
@EunsuSeo01 EunsuSeo01 linked a pull request Nov 17, 2024 that will close this issue
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@EunsuSeo01 EunsuSeo01

Labels
🐬 은수 은수 공주 작업 🛠️ Fix 버그, 오류 해결
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Fix] #289 - 2차 스프린트 기능 QA 반영 Team-Hankki/hankki-iOS
1 participant
@EunsuSeo01