From 8187bb8a38fded41e30fe2ca2d471347b60fdc64 Mon Sep 17 00:00:00 2001 From: Sivin Varghese <64252451+iamsivin@users.noreply.github.com> Date: Mon, 17 Feb 2025 22:37:08 +0530 Subject: [PATCH 01/10] fix: Wrong toast message on creation of a Live Chat campaign (#10919) --- .../CampaignPage/LiveChatCampaign/LiveChatCampaignDialog.vue | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/javascript/dashboard/components-next/Campaigns/Pages/CampaignPage/LiveChatCampaign/LiveChatCampaignDialog.vue b/app/javascript/dashboard/components-next/Campaigns/Pages/CampaignPage/LiveChatCampaign/LiveChatCampaignDialog.vue index 2c4b9848c..ea76c6130 100644 --- a/app/javascript/dashboard/components-next/Campaigns/Pages/CampaignPage/LiveChatCampaign/LiveChatCampaignDialog.vue +++ b/app/javascript/dashboard/components-next/Campaigns/Pages/CampaignPage/LiveChatCampaign/LiveChatCampaignDialog.vue @@ -21,11 +21,11 @@ const addCampaign = async campaignDetails => { type: CAMPAIGN_TYPES.ONGOING, }); - useAlert(t('CAMPAIGN.SMS.CREATE.FORM.API.SUCCESS_MESSAGE')); + useAlert(t('CAMPAIGN.LIVE_CHAT.CREATE.FORM.API.SUCCESS_MESSAGE')); } catch (error) { const errorMessage = error?.response?.message || - t('CAMPAIGN.SMS.CREATE.FORM.API.ERROR_MESSAGE'); + t('CAMPAIGN.LIVE_CHAT.CREATE.FORM.API.ERROR_MESSAGE'); useAlert(errorMessage); } }; From 69e3926db62af1f9e0b8e0911e37f3a16afaa7f6 Mon Sep 17 00:00:00 2001 From: Michael Scrivo <275524+mscrivo@users.noreply.github.com> Date: Tue, 18 Feb 2025 12:03:55 -0500 Subject: [PATCH 02/10] chore(deps): Switch html2text back to rubygems (#10911) ## Description Hi! I've recently started helping maintain this gem as we use it heavily in our app as well. It's been updated to work with newer versions of nokogiri and has a few important fixes too. ## How Has This Been Tested? Assuming you already have test coverage that would cover this. ## Checklist: - [x] My code follows the style guidelines of this project - [x] I have performed a self-review of my code - [x] I have commented on my code, particularly in hard-to-understand areas - [x] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [x] I have added tests that prove my fix is effective or that my feature works - [x] New and existing unit tests pass locally with my changes - [x] Any dependent changes have been merged and published in downstream modules Co-authored-by: Sojan Jose --- Gemfile | 4 +--- Gemfile.lock | 20 +++++++------------- 2 files changed, 8 insertions(+), 16 deletions(-) diff --git a/Gemfile b/Gemfile index 45e194e13..72330acbf 100644 --- a/Gemfile +++ b/Gemfile @@ -138,9 +138,7 @@ gem 'procore-sift' # parse email gem 'email_reply_trimmer' -# TODO: we might have to fork this gem since 0.3.1 has hard depency on nokogir 1.10. -# and this gem hasn't been updated for a while. -gem 'html2text', git: 'https://github.com/chatwoot/html2text_ruby', branch: 'chatwoot' +gem 'html2text' # to calculate working hours gem 'working_hours' diff --git a/Gemfile.lock b/Gemfile.lock index be31e028a..ff0197a90 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -22,14 +22,6 @@ GIT devise (>= 4.0.0, < 5.0.0) railties (>= 5.0.0, < 8.0.0) -GIT - remote: https://github.com/chatwoot/html2text_ruby - revision: cdbdbbbf898d846d0136d69d688a003c6b26074b - branch: chatwoot - specs: - html2text (0.3.1) - nokogiri (>= 1.13.6) - GEM remote: https://rubygems.org/ specs: @@ -361,6 +353,8 @@ GEM hana (1.3.7) hashdiff (1.1.0) hashie (5.0.0) + html2text (0.4.0) + nokogiri (>= 1.0, < 2.0) http (5.1.1) addressable (~> 2.8) http-cookie (~> 1.0) @@ -503,14 +497,14 @@ GEM newrelic_rpm (9.6.0) base64 nio4r (2.7.3) - nokogiri (1.17.1) + nokogiri (1.18.2) mini_portile2 (~> 2.8.2) racc (~> 1.4) - nokogiri (1.17.1-arm64-darwin) + nokogiri (1.18.2-arm64-darwin) racc (~> 1.4) - nokogiri (1.17.1-x86_64-darwin) + nokogiri (1.18.2-x86_64-darwin) racc (~> 1.4) - nokogiri (1.17.1-x86_64-linux) + nokogiri (1.18.2-x86_64-linux-gnu) racc (~> 1.4) oauth (1.1.0) oauth-tty (~> 1.0, >= 1.0.1) @@ -897,7 +891,7 @@ DEPENDENCIES haikunator hairtrigger hashie - html2text! + html2text image_processing jbuilder json_refs From 8a5b007bc47e30380f0368e82900693f1f7b9cba Mon Sep 17 00:00:00 2001 From: Shivam Mishra Date: Wed, 19 Feb 2025 09:07:00 +0530 Subject: [PATCH 03/10] feat: update vue-letter and allow transform css (#10865) Emails from iOS and MacOS clients can often come with size transformation. This results in a broken layout when sanitizing with Vue letter. This PR updates the vue-letter to include the [recent changes made there](https://github.com/mat-sz/vue-letter/pull/2). And adds the `transform` and `transform-origin` properties to the allow list Ref: https://www.caniemail.com/features/css-transform/ --------- Co-authored-by: Muhsin Keloth --- .../message/bubbles/Email/Index.vue | 11 ++++++++++ package.json | 3 ++- pnpm-lock.yaml | 21 +++++++++++-------- 3 files changed, 25 insertions(+), 10 deletions(-) diff --git a/app/javascript/dashboard/components-next/message/bubbles/Email/Index.vue b/app/javascript/dashboard/components-next/message/bubbles/Email/Index.vue index ea2f57f34..05031f055 100644 --- a/app/javascript/dashboard/components-next/message/bubbles/Email/Index.vue +++ b/app/javascript/dashboard/components-next/message/bubbles/Email/Index.vue @@ -1,6 +1,7 @@ +