Skip to content

[CLNP-8133][Fix]Fixed a bug where the markAsUnread menu was not displayed correctly w…#1400

Open
danney-chun wants to merge 2 commits intomainfrom
bug-fix/CLNP-8133
Open

[CLNP-8133][Fix]Fixed a bug where the markAsUnread menu was not displayed correctly w…#1400
danney-chun wants to merge 2 commits intomainfrom
bug-fix/CLNP-8133

Conversation

@danney-chun
Copy link
Contributor

@danney-chun danney-chun commented Feb 5, 2026

Fixes CLNP-8133

Changelogs

  • Fixed a bug where the markAsUnread menu was not displayed correctly when replyType was set to thread

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If unsure, ask the members.
This is a reminder of what we look for before merging your code.

  • All tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • Public components / utils / props are appropriately exported
  • I have added necessary documentation (if appropriate)

External Contributions

This project is not yet set up to accept pull requests from external contributors.

If you have a pull request that you believe should be accepted, please contact
the Developer Relations team developer-advocates@sendbird.com with details
and we'll evaluate if we can set up a CLA to allow for the contribution.

@netlify
Copy link

netlify bot commented Feb 5, 2026

Deploy Preview for sendbird-uikit-react ready!

Name Link
🔨 Latest commit a1b28ef
🔍 Latest deploy log https://app.netlify.com/projects/sendbird-uikit-react/deploys/69842bfd50b7060008799efa
😎 Deploy Preview https://deploy-preview-1400--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 project configuration.

@danney-chun danney-chun self-assigned this Feb 5, 2026
disableDeleteMessage?: boolean;
showEdit?: (bool: boolean) => void;
showRemove?: (bool: boolean) => void;
showMarkAsUnread?: (bool: boolean) => void;
Copy link
Contributor

Choose a reason for hiding this comment

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

이거 추가된건 어떤용도일까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

앗. 사용을 하지 않습니다.
삭제 했습니다.

onMoveToParentMessage,
};

console.log('MADOKA inThreadList', inThreadList);
Copy link
Contributor

Choose a reason for hiding this comment

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

로그 삭제 해야될것 같습니다

Copy link
Contributor Author

Choose a reason for hiding this comment

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

삭제 했습니다.

Copy link
Contributor

@OnestarLee OnestarLee left a comment

Choose a reason for hiding this comment

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

lgtm!

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.

2 participants