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

[mod] 핑글 개최 프로세스 - 카테고리 선택 뷰 수정 #62

Merged
merged 1 commit into from
Jan 9, 2024

Conversation

HAJIEUN02
Copy link
Collaborator

Related issue 🛠

Work Description ✏️

  • viewModel 네이밍 수정
  • setSelectedCategory 추가 및 viewModel 저장 확인

Screenshot 📸

초기 구현과 같으므로 생략합니다.

Uncompleted Tasks 😅

  • 버튼 활성화 로직은 마지막에

To Reviewers 📢

제가 viewModel이름을 구데기같이 해서 안 됐던 것 같습니다 하나하나 브랜치별로 고치는 중이에요..
그리구 마지막에 뷰모델에 저장된 값들을 모두 출력하려고 보니 제대로 저장이 안되는듯해서 확인해보니 제가 구현을 안해둔 부분이 있더라구요!! 팀버 찍어서 확인해보고 지우고 올립니다.
바뀐 파일 하나 뿐이니 어프룹만 부탁드려염ㅋㅅㅋ.

@HAJIEUN02 HAJIEUN02 added this to the 1차 스프린트 뷰 작업 milestone Jan 8, 2024
@HAJIEUN02 HAJIEUN02 self-assigned this Jan 8, 2024
@HAJIEUN02 HAJIEUN02 requested a review from a team as a code owner January 8, 2024 19:53
@HAJIEUN02 HAJIEUN02 changed the title [feat] 핑글 개최 프로세스 - 카테고리 선택 뷰 수 [mod] 핑글 개최 프로세스 - 카테고리 선택 뷰 수정 Jan 8, 2024
Copy link
Collaborator

@jihyunniiii jihyunniiii left a comment

Choose a reason for hiding this comment

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

조아요 ~
근데 mod 할 때는 브랜치 이름 mod로 시작해주고 (컨벤션임)
PR과 이슈 이름에 무엇을 수정하셨는지를 자세하게 적어주세요

HAJIEUN02 added a commit that referenced this pull request Jan 9, 2024
HAJIEUN02 added a commit that referenced this pull request Jan 9, 2024
@HAJIEUN02 HAJIEUN02 merged commit 8d4f257 into develop Jan 9, 2024
1 check passed
@HAJIEUN02 HAJIEUN02 deleted the feat-plan-category branch January 12, 2024 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[mod] 핑글 개최 프로세스 - 모집인원 입력 뷰 수정
2 participants