chore: Fix link generation logic in help-center (#5470)

app/controllers/public/api/v1/portals_controller.rb

@@ -1,7 +1,7 @@
 class Public::Api::V1::PortalsController < PublicController
   before_action :ensure_custom_domain_request, only: [:show]
-  before_action :redirect_to_portal_with_locale, only: [:show]
   before_action :portal
+  before_action :redirect_to_portal_with_locale, only: [:show]
   layout 'portal'
 
   def show; end

app/javascript/dashboard/helper/portalHelper.js

@@ -4,7 +4,12 @@ export const buildPortalURL = portalSlug => {
   return `${baseURL}/hc/${portalSlug}`;
 };
 
-export const buildPortalArticleURL = (portalSlug, categorySlug, articleId) => {
+export const buildPortalArticleURL = (
+  portalSlug,
+  categorySlug,
+  locale,
+  articleId
+) => {
   const portalURL = buildPortalURL(portalSlug);
-  return `${portalURL}/${categorySlug}/${articleId}`;
+  return `${portalURL}/${locale}/${categorySlug}/${articleId}`;
 };

app/javascript/dashboard/helper/specs/portalHelper.spec.js

@@ -0,0 +1,29 @@
+import { buildPortalArticleURL, buildPortalURL } from '../portalHelper';
+
+describe('PortalHelper', () => {
+  describe('buildPortalURL', () => {
+    it('returns the correct url', () => {
+      window.chatwootConfig = {
+        hostURL: 'https://app.chatwoot.com',
+        helpCenterURL: 'https://help.chatwoot.com',
+      };
+      expect(buildPortalURL('handbook')).toEqual(
+        'https://help.chatwoot.com/hc/handbook'
+      );
+      window.chatwootConfig = {};
+    });
+  });
+
+  describe('buildPortalArticleURL', () => {
+    it('returns the correct url', () => {
+      window.chatwootConfig = {
+        hostURL: 'https://app.chatwoot.com',
+        helpCenterURL: 'https://help.chatwoot.com',
+      };
+      expect(buildPortalArticleURL('handbook', 'culture', 'fr', 1)).toEqual(
+        'https://help.chatwoot.com/hc/handbook/fr/culture/1'
+      );
+      window.chatwootConfig = {};
+    });
+  });
+});

app/javascript/dashboard/i18n/locale/en/conversation.json

@@ -1,6 +1,7 @@
 {
   "CONVERSATION": {
     "SELECT_A_CONVERSATION": "Please select a conversation from left pane",
+    "CSAT_REPLY_MESSAGE": "Please rate the conversation",
     "404": "Sorry, we cannot find the conversation. Please try again",
     "SWITCH_VIEW_LAYOUT": "Switch the layout",
     "DASHBOARD_APP_TAB_MESSAGES": "Messages",

app/javascript/dashboard/routes/dashboard/helpcenter/pages/articles/EditArticle.vue

@@ -90,6 +90,7 @@ export default {
       return buildPortalArticleURL(
         slug,
         this.article.category.slug,
+        this.article.category.locale,
         this.article.id
       );
     },

app/views/api/v1/accounts/articles/_article.json.jbuilder

@@ -10,6 +10,7 @@ json.category do
   json.id article.category_id
   json.name article.category.name
   json.slug article.category.slug
+  json.locale article.category.locale
 end
 
 if article.portal.present?

app/views/public/api/v1/portals/_category-block.html.erb

@@ -15,7 +15,7 @@
     </div>
     <% else %>
       <% category.articles.published.take(5).each do |article| %>
-      <div class="flex justify-between content-center min:h-8 my-1">
+      <div class="flex justify-between content-center my-1">
         <a
           class="text-slate-800 hover:underline leading-8"
           href="/hc/<%= portal.slug %>/<%= category.locale %>/<%= category.slug %>/<%= article.id %>"

app/views/public/api/v1/portals/_header.html.erb

@@ -17,6 +17,7 @@
       </div>
       <% end %>
     </div>
+    <% if @portal.config["allowed_locales"].length > 1 %>
     <div class="flex items-center">
       <div class="inline-flex relative w-24">
         <select
@@ -32,5 +33,6 @@
         </div>
       </div>
     </div>
+    <% end %>
   </nav>
 </header>