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

Chip 파트 한 개 이슈 대응 #932

Merged
merged 16 commits into from
Oct 13, 2024
Merged

Chip 파트 한 개 이슈 대응 #932

merged 16 commits into from
Oct 13, 2024

Conversation

j-nary
Copy link
Member

@j-nary j-nary commented Oct 13, 2024

🚩 관련 이슈

📋 작업 내용

  • Chip 파트 한 개 이슈 대응

📌 PR Point

  • 무슨 이유로 어떻게 코드를 변경했는지
  • 어떤 위험이나 우려가 발견되었는지
  • 어떤 부분에 리뷰어가 집중해야 하는지
  • 개발하면서 어떤 점이 궁금했는지

📸 스크린샷

@j-nary j-nary self-assigned this Oct 13, 2024
@height Height
Copy link

height bot commented Oct 13, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@j-nary j-nary merged commit 78d7678 into develop Oct 13, 2024
2 checks passed
j-nary added a commit that referenced this pull request Oct 21, 2024
@j-nary
Chip, dialog 배포 (#938)
16e78ef 
* Chip 파트 한 개 이슈 대응 (#932)

* test: 임시로 Chip 구현 (MDS 문의 필요)

* feat: chip 구현 및 반응형 작업

* feat: 전체파트 클릭 로직 + MDS Chip 연결

* docs: 불필요한 파일 삭제

* docs: 불필요한 코드 삭제

* fix: 개별 옵션 해제 시 전체파트 해제 로직 추가

* refactor: 코드리뷰 반영

* refactor: input 높이 반응형 제거

* feat: 모임 안내 모집 대상 chip 변경

* style: 색상변경

* style: QA 반영

* fix: 이슈대응

* fix: Chip 한개 선택 이슈 대응

* fix: Chip 5개 선택 시 이슈 해결 (#934)

* feat: 모임 삭제 dialog mds 마이그레이션 (#937)
j-nary added a commit that referenced this pull request Nov 13, 2024
@j-nary @borimong
API 마이그레이션 마무리 (#953)
480c484 
* Chip, dialog 배포 (#938)

* Chip 파트 한 개 이슈 대응 (#932)

* test: 임시로 Chip 구현 (MDS 문의 필요)

* feat: chip 구현 및 반응형 작업

* feat: 전체파트 클릭 로직 + MDS Chip 연결

* docs: 불필요한 파일 삭제

* docs: 불필요한 코드 삭제

* fix: 개별 옵션 해제 시 전체파트 해제 로직 추가

* refactor: 코드리뷰 반영

* refactor: input 높이 반응형 제거

* feat: 모임 안내 모집 대상 chip 변경

* style: 색상변경

* style: QA 반영

* fix: 이슈대응

* fix: Chip 한개 선택 이슈 대응

* fix: Chip 5개 선택 시 이슈 해결 (#934)

* feat: 모임 삭제 dialog mds 마이그레이션 (#937)

* Update FeedFormPresentation.tsx

* Update FeedFormPresentation.tsx

* refactor: 모임 삭제

* refactor: 모임 지원자 상태 변경

* refactor: 로그인

* 모임삭제 코리반영

---------

Co-authored-by: Hyeonsu Kim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@borimong borimong Awaiting requested review from borimong borimong is a code owner

@ocahs9 ocahs9 Awaiting requested review from ocahs9 ocahs9 is a code owner

Assignees

@j-nary j-nary

Labels
size/XS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Chip 이슈 대응
1 participant
@j-nary