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

[SAMBAD-258] release/1.0.0-beta.4 rebase 업데이트 #110

Closed
wants to merge 2 commits into from

Conversation

nahyeon99
Copy link
Member

@nahyeon99 nahyeon99 commented Aug 17, 2024

✔️ PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

📝 개요

nahyeon99 and others added 2 commits August 17, 2024 16:11
* refactor: 종료된 모임 질문의 경우 참여율 계산 로직 수정

* fix: 계산 메서드 로직 오류 수정

* refactor: 비활성화된 질문의 비활성화 시도 시 대응 코드 추가

* fix: 질문 개수 범위 오류 수정

* refactor: 전체 모임원 수 null 비허용하도록 수정
* [SAMBAD-254]-feat: Poking 엔티티 작성

* [SAMBAD-254]-feat: Poking controller service repository 추가

* [SAMBAD-254]-refactor: poking에서 handWaving으로 변경
@nahyeon99 nahyeon99 requested review from kkjsw17 and LeeHanEum August 17, 2024 07:40
@nahyeon99 nahyeon99 self-assigned this Aug 17, 2024
Copy link

Unit Test Results

4 tests   4 ✔️  0s ⏱️
1 suites  0 💤
1 files    0

Results for commit 0bb7a6c.

Copy link
Collaborator

@kkjsw17 kkjsw17 left a comment

Choose a reason for hiding this comment

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

이제 봤네요! release/1.0.0 브랜치를 develop에 rebase merge하는 방식이 되어야 할 것 같아요.

@kkjsw17
Copy link
Collaborator

kkjsw17 commented Aug 19, 2024

스크린샷 2024-08-19 오후 2 41 25

여기서 SAMBAD-255 커밋만 develop에 머지가 되어야 하는 상태이고,
rebase 진행하면 254 커밋 밑에 255 커밋이 들어가야 하는 형태입니다.

@nahyeon99
Copy link
Member Author

@kkjsw17 혹시 몰라서 다시 작업해봤는데, 여전히 2개 커밋 들어가게 되네요 ㅠㅠ 제가 어떤 부분을 놓쳤을까요?

스크린샷 2024-08-19 오후 6 20 12

@kkjsw17
Copy link
Collaborator

kkjsw17 commented Aug 19, 2024

@kkjsw17 혹시 몰라서 다시 작업해봤는데, 여전히 2개 커밋 들어가게 되네요 ㅠㅠ 제가 어떤 부분을 놓쳤을까요?

스크린샷 2024-08-19 오후 6 20 12

이따 게더에서 같이 확인해봅시당

@nahyeon99 nahyeon99 closed this Aug 19, 2024
@nahyeon99 nahyeon99 deleted the release/SAMBAD-258-sync_update_to_develop branch August 19, 2024 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants