Skip to content

Conversation

@caniro
Copy link
Contributor

@caniro caniro commented Jun 7, 2025

📝 요약(Summary)

  • 백엔드 체결 알림 추가했는데 프론트 일정 빠듯하실 것 같아서 일단 화면에 보이는 정도로만 작성했습니다.

🛠️ PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가

💬 공유사항 to 리뷰어

  • 알림이 전역적인 기능이라 root에 넣긴 했는데 뭔가 찝찝해서.. 일관성 있게 리팩토링 부탁드려요...
  • 아래처럼 알림 많아지면 화면을 가리는데 이거 맞나요..? ㅋㅋㅋ

📸스크린샷

image


## ✅ PR Checklist PR이 다음 요구 사항을 충족하는지 확인하세요.
  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

@BHyeonKim
Copy link
Contributor

감사합니다. 합친후에 리팩토링 하겠습니다!

@investfuture-sonarqube1
Copy link

Failed Quality Gate failed

  • 1 New Issues (is greater than 0)
  • 0.00% Coverage on New Code (is less than 80.00%)

Project ID: CleanEngine_cleanengine-fe_c6875537-ed9d-4dfe-b79c-ea5cb6d2a0e3

View in SonarQube

@BHyeonKim BHyeonKim merged commit 2b28f25 into develop Jun 8, 2025
1 check passed
@caniro caniro deleted the feat/trade-notify branch June 8, 2025 14:12
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