Skip to content

Conversation

@git-babel
Copy link
Contributor

@git-babel git-babel commented Feb 14, 2025

SBISSUE-18534

Changelog

  • Fixed the bug where VoiceMessageIssue doesn't work at the first time.

@git-babel git-babel requested a review from chrisallo February 14, 2025 04:06
@netlify
Copy link

netlify bot commented Feb 14, 2025

Deploy Preview for sendbird-uikit-react ready!

Name Link
🔨 Latest commit 7f5d4a9
🔍 Latest deploy log https://app.netlify.com/sites/sendbird-uikit-react/deploys/67aed1c84f15a10009b440eb
😎 Deploy Preview https://deploy-preview-1311--sendbird-uikit-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

}
};
const handleOnControlClick = useCallback(() => {
const handleOnControlClick = () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기서 useCallback을 없내는것도 방법이지만 dependencies에 lastClickTime, onControlClick, setLastClickTime을 추가하는것도 방법이 될 수 있을것 같습니다. 기존에도 아마 lastClickTime이 바뀌었을때에도 오동작을 하고 있었을것같네요.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

추후 Generelize해서 논의하는 걸로 결정되었습니다.

Copy link
Collaborator

@chrisallo chrisallo left a comment

Choose a reason for hiding this comment

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

lgtm!

@git-babel git-babel merged commit 6c31493 into main Feb 14, 2025
10 checks passed
@bang9 bang9 deleted the fix/sbissue-18534 branch November 7, 2025 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants