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

[Fix] Coffee Chat QA 사항 반영 #532

Merged
merged 3 commits into from
Nov 6, 2024
Merged

[Fix] Coffee Chat QA 사항 반영 #532

merged 3 commits into from
Nov 6, 2024

Conversation

jun02160
Copy link
Member

@jun02160 jun02160 commented Nov 5, 2024

🐬 요약

PR의 요약을 작성해주세요!

👻 유형

PR의 유형에 맞게 체크해주세요!

  • 버그 수정
  • 기능 개발
  • 코드 스타일 수정 (formatting, local variables)
  • 리팩토링 (no functional changes, no api changes)
  • 빌드 관련 변경사항
  • CI 관련 변경사항
  • CD 관련 변경사항
  • 문서 내용 변경
  • Release
  • 기타... (다음 줄에 사유를 입력해주세요)

🍀 작업 내용

PR에 담긴 작업 내용을 작성해주세요!

  • Coffee Chat Request topic validation 200 -> 1000자 로 수정
  • Coffee Chat 문자 발송 시, 기수 정보에 [ ] 문자 제외하고 전송하도록 String 변환 추가

🌟 관련 이슈

PR과 관련된 이슈 번호를 작성해주세요!

close: #531

@jun02160 jun02160 added ⛏ Fix 버그, 오류 해결 ✅ Chore 자잘한 수정 시 사용합니다. 예준🐈 예준이 연 PR, Issue에 사용 labels Nov 5, 2024
@jun02160 jun02160 requested a review from dev-Crayon November 5, 2024 14:27
@jun02160 jun02160 self-assigned this Nov 5, 2024
Copy link

height bot commented Nov 5, 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.

@@ -9,6 +9,7 @@
import org.springframework.stereotype.Component;

import java.util.List;
import java.util.stream.Collectors;
Copy link
Contributor

Choose a reason for hiding this comment

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

혹시 해당 Import가 추가된 이유가 있을까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

앗 테스트 중에 추가된 것 같네요
1acea81 에서 반영했습니다!

- remove unused import
@jun02160 jun02160 merged commit 1e21179 into develop Nov 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅ Chore 자잘한 수정 시 사용합니다. ⛏ Fix 버그, 오류 해결 size/XS 예준🐈 예준이 연 PR, Issue에 사용
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 커피챗 QA 반영
2 participants