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] 리뷰 작성 에러 해결 #135

Merged
merged 4 commits into from
Sep 27, 2024
Merged

[Fix] 리뷰 작성 에러 해결 #135

merged 4 commits into from
Sep 27, 2024

Conversation

binisnil
Copy link
Collaborator

🍎 EATSSU iOS Team Pull Request

🔆 개요

  • #56에서 확인되는 버그를 해결했습니다

💡 PR Point

새로운 다중 리뷰 작성하기 로직은 아래와 같습니다

  • selectedIDList, selectedList에 각각 선택한 메뉴의 ID와 메뉴명을 넣어둠
  • currentPage 변수를 만듦(현재 몇번째 메뉴를 리뷰하고 있는지 파악하기 위함)
  • reviewList(리뷰 작성 API DTO타입)를 만들어서, reviewList[currentPage]에 리뷰 내용을 저장

📷 스크린샷

테스트 영상이 길어서 슬랙으로 첨부하겠습니다~

📮 관련 이슈

@binisnil binisnil self-assigned this Sep 26, 2024
@jayn2u jayn2u self-requested a review September 27, 2024 02:03
@jayn2u jayn2u merged commit 0006d55 into develop Sep 27, 2024
@jayn2u jayn2u deleted the fix/#56 branch September 27, 2024 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jayn2u jayn2u jayn2u approved these changes

@CJiu01 CJiu01 Awaiting requested review from CJiu01

Assignees

@binisnil binisnil

Labels
🛠️ Fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Fix] 리뷰 중복 작성됨
2 participants
@binisnil @jayn2u