Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(misunderstood): add support for all types of messages #5731

Merged
merged 8 commits into from
Dec 10, 2021

Conversation

laurentlp
Copy link
Contributor

@laurentlp laurentlp commented Dec 2, 2021

Description

This PR adds support for all types of messaging to the misunderstood module by using the @botpress/messaging-components library.

Closes DEV-2077

Type of change

  • New feature (non-breaking change which adds functionality)

Missing

  • Fix Card component style

Screenshots

Screenshot from 2021-12-02 18-23-54
Screenshot from 2021-12-02 18-23-40
Screenshot from 2021-12-02 18-23-24

@linear
Copy link

linear bot commented Dec 2, 2021

DEV-2077 [FEATURE] Misunderstood - Custom component support

We currently don't support custom component in the ui messages.

Who ever is going to work on this, custom components are located in the window element.

border: 1px solid #ccc;

p {
margin: 0;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Somehow bootstrap was adding a margin to the p element that was causing the text inside each bubbles to be wrongly aligned.

@laurentlp laurentlp force-pushed the llp_misunderstood_use_messaging_components branch from 3fa526e to 53a4ee7 Compare December 2, 2021 18:41
@laurentlp laurentlp changed the title feat(misunderstood): add support for all content-types feat(misunderstood): add support for all types of messages Dec 2, 2021
samuelmasse
samuelmasse previously approved these changes Dec 6, 2021
Copy link
Contributor

@samuelmasse samuelmasse left a comment

Choose a reason for hiding this comment

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

Tested and working good job 👍

Comment on lines +25 to +28
// TODO: Add translation and move this logic into the @botpress/messaging-components lib
if (message.type === 'session_reset') {
return <div className={style.chatPreviewMessage_System}>Reset the conversation</div>
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this todo still valid?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@samuelmasse I think so. It might be nice to have a way to display conversation reset using the library. What do you think?

@laurentlp laurentlp merged commit 8c6adff into master Dec 10, 2021
@laurentlp laurentlp deleted the llp_misunderstood_use_messaging_components branch December 10, 2021 22:09
@EFF EFF mentioned this pull request Jan 17, 2022
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.

None yet

3 participants