From 035d766886fd6b4bcc7a238be342e5162233d553 Mon Sep 17 00:00:00 2001
From: Awais Ansari <79941147+awais-ansari@users.noreply.github.com>
Date: Tue, 9 Jan 2024 12:30:18 +0500
Subject: [PATCH] feat: removed sort and filter tour (#638)

---
 .../post-comments/comments/CommentsSort.jsx   |  1 -
 .../posts/post-filter-bar/PostFilterBar.jsx   |  2 +-
 .../tours/DiscussionsProductTour.jsx          | 13 ++++------
 src/discussions/tours/constants.js            | 23 +++++++---------
 src/discussions/tours/messages.js             | 26 ++++++-------------
 5 files changed, 24 insertions(+), 41 deletions(-)

diff --git a/src/discussions/post-comments/comments/CommentsSort.jsx b/src/discussions/post-comments/comments/CommentsSort.jsx
index bebaecb03..fd59e55ed 100644
--- a/src/discussions/post-comments/comments/CommentsSort.jsx
+++ b/src/discussions/post-comments/comments/CommentsSort.jsx
@@ -44,7 +44,6 @@ const CommentSortDropdown = () => {
     <>
       <div className="comments-sort d-flex justify-content-end mx-4 mt-2">
         <Button
-          id="comment-sort"
           alt={intl.formatMessage(messages.actionsAlt)}
           ref={setTarget}
           variant="tertiary"
diff --git a/src/discussions/posts/post-filter-bar/PostFilterBar.jsx b/src/discussions/posts/post-filter-bar/PostFilterBar.jsx
index 3633096e1..040ae4ef0 100644
--- a/src/discussions/posts/post-filter-bar/PostFilterBar.jsx
+++ b/src/discussions/posts/post-filter-bar/PostFilterBar.jsx
@@ -197,7 +197,7 @@ const PostFilterBar = () => {
             cohort: capitalize(selectedCohort?.name),
           })}
         </span>
-        <span id="icon-tune">
+        <span>
           <Collapsible.Visible whenClosed>
             <Icon src={Tune} />
           </Collapsible.Visible>
diff --git a/src/discussions/tours/DiscussionsProductTour.jsx b/src/discussions/tours/DiscussionsProductTour.jsx
index 50475af5a..aa9b741b9 100644
--- a/src/discussions/tours/DiscussionsProductTour.jsx
+++ b/src/discussions/tours/DiscussionsProductTour.jsx
@@ -17,14 +17,11 @@ const DiscussionsProductTour = () => {
   }, []);
 
   return (
-    // eslint-disable-next-line react/jsx-no-useless-fragment
-    <>
-      {!isEmpty(config) && (
-        <ProductTour
-          tours={config}
-        />
-      )}
-    </>
+    !isEmpty(config) && (
+      <ProductTour
+        tours={config}
+      />
+    )
   );
 };
 
diff --git a/src/discussions/tours/constants.js b/src/discussions/tours/constants.js
index e1129c438..8d7949dd4 100644
--- a/src/discussions/tours/constants.js
+++ b/src/discussions/tours/constants.js
@@ -1,21 +1,18 @@
 import messages from './messages';
 
+/**
+ *
+ * @param {Object} intl
+ * @returns {Object} tour checkpoints
+ */
 export default function tourCheckpoints(intl) {
   return {
-    NOT_RESPONDED_FILTER: [
+    EXAMPLE_TOUR: [
       {
-        body: intl.formatMessage(messages.notRespondedFilterTourBody),
-        placement: 'right',
-        target: '#icon-tune',
-        title: intl.formatMessage(messages.notRespondedFilterTourTitle),
-      },
-    ],
-    RESPONSE_SORT: [
-      {
-        body: intl.formatMessage(messages.responseSortTourBody),
-        placement: 'left',
-        target: '#comment-sort',
-        title: intl.formatMessage(messages.responseSortTourTitle),
+        title: intl.formatMessage(messages.exampleTourTitle),
+        body: intl.formatMessage(messages.exampleTourBody),
+        target: '#example-tour-target',
+        placement: 'bottom',
       },
     ],
   };
diff --git a/src/discussions/tours/messages.js b/src/discussions/tours/messages.js
index d693e5216..f65c2af22 100644
--- a/src/discussions/tours/messages.js
+++ b/src/discussions/tours/messages.js
@@ -16,25 +16,15 @@ const messages = defineMessages({
     defaultMessage: 'Okay',
     description: 'Action to end current tour',
   },
-  notRespondedFilterTourBody: {
-    id: 'tour.body.notRespondedFilter',
-    defaultMessage: 'Now you can filter discussions to find posts with no response.',
-    description: 'Body of the tour for the not responded filter',
+  exampleTourTitle: {
+    id: 'tour.example.title',
+    defaultMessage: 'Example Tour',
+    description: 'Title for example tour',
   },
-  notRespondedFilterTourTitle: {
-    id: 'tour.title.notRespondedFilter',
-    defaultMessage: 'New filtering option!',
-    description: 'Title of the tour for the not responded filter',
-  },
-  responseSortTourBody: {
-    id: 'tour.body.responseSortTour',
-    defaultMessage: 'Responses and comments are now sorted by newest first. Please use this option to change the sort order',
-    description: 'Body of the tour for the response sort',
-  },
-  responseSortTourTitle: {
-    id: 'tour.title.responseSortTour',
-    defaultMessage: 'Sort Responses!',
-    description: 'Title of the tour for the response sort',
+  exampleTourBody: {
+    id: 'tour.example.body',
+    defaultMessage: 'This is an example tour',
+    description: 'Body for example tour',
   },
 });