Skip to content

Conversation

@HoonBaek
Copy link
Contributor

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 setup a CLA to allow for the contribution.

For Internal Contributors

QU-309

Description Of Changes

Types Of Changes

What types of changes does your code introduce to this project?
Put an x in the boxes that apply_

  • Bugfix
  • New feature
  • Documentation (correction or otherwise)
  • Cosmetics (whitespace, appearance (ex) Prettier)
  • Build configuration
  • Improvement (refactor code)

@HoonBaek HoonBaek requested a review from sravan-s October 26, 2021 06:09
@HoonBaek HoonBaek self-assigned this Oct 26, 2021
Copy link
Contributor

@sravan-s sravan-s left a comment

Choose a reason for hiding this comment

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

This is okay

@HoonBaek HoonBaek merged commit 7970a8e into main Oct 26, 2021
@HoonBaek HoonBaek deleted the bugfix/QU-309/Check-message-id-when-updating branch October 26, 2021 23:33
HoonBaek added a commit that referenced this pull request Nov 4, 2021
* Remove sendbird.min.js file and import released core sdk

* Update snapshot

* Fix message list params filter condition about customTypes

* [QU-309] Do not update last message if message id doesn't match (#47)

* [QU-299] Fix layout of QuoteMesssageInput component when it is file message

* Add channel event management logic for onThreadInfoUpdated

* [QU-336][1] Remove the gap created by the minus bottom position

* [QU-336][2] Fix layout of message input in the message input wrapper component

* Fix message list params condition

* Use optional chaning in js

* Add test codes for ON_MESSAGE_THREAD_INFO_UPDATE event
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