-
Notifications
You must be signed in to change notification settings - Fork 72
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
Update Twilio Conversations & Paste to latest #27
base: main
Are you sure you want to change the base?
Conversation
…ste and dependencies, updated tests to pass
…ste and dependencies, updated tests to pass
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Just had a note about the indentation level 👍
…ste and dependencies, updated tests to pass
@nayanadasgupta I also merged the latest changes from main fork, it took a couple of commits because I missed to resolve conflicts in a file. Let me know if they look ok because prev file's format on some do not match the prettier policy even before my commits. |
export interface ParticipantResponse { | ||
account_sid: string; | ||
chat_service_sid: string; | ||
conversation_sid: string; | ||
role_sid: string; | ||
sid: string; | ||
attributes: string; | ||
date_created: string; | ||
date_updated: string; | ||
identity: string; | ||
messaging_binding: { | ||
type: "chat" | "sms" | "whatsapp" | "email"; | ||
address: string; | ||
proxy_address: string; | ||
} | null; | ||
url: string; | ||
links: { | ||
conversation: string; | ||
}; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wondering if it's necessary to define this purely for the mock file's benefit. Was this because the new Conversations version required it?
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
This pull request is for updating the Twilio Conversations library to the latest stable one and Twilio Paste (new design system) to latest version for improved stability.
Also, some tests have been updated to account for Pastes current state regarding components. This includes a custom query for locking for the special data-paste-element attribute used along Paste's components.
Contributing to Twilio