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

UK-973

Description Of Changes

Add a brief description of the changes that you have involved in this PR

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 13, 2021 01:47
@HoonBaek HoonBaek self-assigned this Oct 13, 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.

src/sendbird.min.js
shouldnt be here, right?

@HoonBaek HoonBaek requested a review from sravan-s October 14, 2021 02:47
&.menu-item-edit {}
&.menu-item-resend {}
&.menu-item-delete {}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

lets remove it when we merge to main

@HoonBaek HoonBaek merged commit 93e3f77 into feature/UK-932/Integrate-message-threading Oct 14, 2021
@HoonBaek HoonBaek deleted the bugfix/UK-973/Design-change-for-message-menu-and-status branch October 14, 2021 08:11
HoonBaek added a commit that referenced this pull request Oct 25, 2021
* Add class names of MessageItemMenu

* Change the message status design to row

* Apply that replyTypeFilter is changed to replyType in core sdk

* Add onLoad and onError props in ImageRenderer component

* Change opacity and icon display of QuoteMessage

* Move QuoteMessage into the MessageContent

* Fix the center align of IconButton

* Update core min.js file

* Update snapshot

* Update class name of menu items
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