Skip to content

Conversation

@yunsaeng
Copy link
Contributor

Pull requests

작업한 내용

  • 작업한 내용을 적어주세요.

PR Point

  • 코드 리뷰 시 중점적으로 봐야할 부분에 대해 설명해주세요.

📸 스크린샷

  • 스크린 샷 혹은 동영상을 첨부해주세요.

closed #이슈번호

@yunsaeng yunsaeng self-assigned this Nov 20, 2024
@yunsaeng yunsaeng linked an issue Nov 20, 2024 that may be closed by this pull request
3 tasks
@netlify
Copy link

netlify bot commented Nov 20, 2024

Deploy Preview for coach-coach ready!

Name Link
🔨 Latest commit 08f7e52
🔍 Latest deploy log https://app.netlify.com/sites/coach-coach/deploys/673e5a1b8ba1760008a5873d
😎 Deploy Preview https://deploy-preview-397--coach-coach.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 62
Accessibility: 76
Best Practices: 100
SEO: 75
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

}
};
setMessages(initialMessages); // 기존 메시지 가져오기
}, [chatRoomId]);

Choose a reason for hiding this comment

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

⚠️ [eslint] <react-hooks/exhaustive-deps> reported by reviewdog 🐶
React Hook useEffect has a missing dependency: 'initialMessages'. Either include it or remove the dependency array.

@yunsaeng yunsaeng merged commit e81b7c2 into main Nov 20, 2024
5 checks passed
@yunsaeng yunsaeng deleted the fix/likes-ask branch November 20, 2024 21:53
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.

FIx: 좋아요 & 문의하기

2 participants