DEV: Remove the legacy widget post menu code

Author: megothss

assets/javascripts/discourse/components/assign-button.gjs

@@ -29,11 +29,17 @@ export default class AssignButton extends Component {
   }
 
   @action
-  acceptAnswer() {
+  async acceptAnswer() {
     if (this.isAssigned) {
-      unassignPost(this.args.post, this.taskActions);
+      const post = this.args.post;
+
+      await this.taskActions.unassign(post.id, "Post");
+      delete post.topic.indirectly_assigned_to[post.id];
     } else {
-      assignPost(this.args.post, this.taskActions);
+      this.taskActions.showAssignModal(this.args.post, {
+        isAssigned: false,
+        targetType: "Post",
+      });
     }
   }
 
@@ -51,16 +57,3 @@ export default class AssignButton extends Component {
     />
   </template>
 }
-
-// TODO (glimmer-post-menu): Remove these exported functions and move the code into the button action after the widget code is removed
-export function assignPost(post, taskActions) {
-  taskActions.showAssignModal(post, {
-    isAssigned: false,
-    targetType: "Post",
-  });
-}
-
-export async function unassignPost(post, taskActions) {
-  await taskActions.unassign(post.id, "Post");
-  delete post.topic.indirectly_assigned_to[post.id];
-}

assets/javascripts/discourse/initializers/extend-for-assigns.js

@@ -778,7 +778,7 @@ function initialize(api) {
 }
 
 function customizePostMenu(api) {
-  const transformerRegistered = api.registerValueTransformer(
+  api.registerValueTransformer(
     "post-menu-buttons",
     ({
       value: dag,
@@ -804,47 +804,6 @@ function customizePostMenu(api) {
       );
     }
   );
-
-  const silencedKey =
-    transformerRegistered && "discourse.post-menu-widget-overrides";
-
-  withSilencedDeprecations(silencedKey, () => customizeWidgetPostMenu(api));
-}
-
-function customizeWidgetPostMenu(api) {
-  api.addPostMenuButton("assign", (post) => {
-    if (post.firstPost) {
-      return;
-    }
-    if (post.assigned_to_user || post.assigned_to_group) {
-      return {
-        action: "unassignPost",
-        icon: "user-xmark",
-        className: "unassign-post",
-        title: "discourse_assign.unassign_post.title",
-        position:
-          post.assigned_to_user?.id === api.getCurrentUser().id
-            ? "first"
-            : "second-last-hidden",
-      };
-    } else {
-      return {
-        action: "assignPost",
-        icon: "user-plus",
-        className: "assign-post",
-        title: "discourse_assign.assign_post.title",
-        position: "second-last-hidden",
-      };
-    }
-  });
-
-  api.attachWidgetAction("post", "assignPost", function () {
-    assignPost(this.model, getOwner(this).lookup("service:task-actions"));
-  });
-
-  api.attachWidgetAction("post", "unassignPost", function () {
-    unassignPost(this.model, getOwner(this).lookup("service:task-actions"));
-  });
 }
 
 const REGEXP_USERNAME_PREFIX = /^(assigned:)/gi;

test/javascripts/acceptance/assign-enabled-test.js

@@ -15,7 +15,7 @@ import selectKit from "discourse/tests/helpers/select-kit-helper";
 acceptance("Discourse Assign | Assign mobile", function (needs) {
   needs.user();
   needs.mobileView();
-  needs.settings({ glimmer_post_menu_mode: "enabled", assign_enabled: true });
+  needs.settings({ assign_enabled: true });
 
   needs.pretender((server, helper) => {
     server.get("/assign/suggestions", () => {
@@ -47,7 +47,7 @@ acceptance("Discourse Assign | Assign mobile", function (needs) {
 
 acceptance("Discourse Assign | Assign desktop", function (needs) {
   needs.user({ can_assign: true });
-  needs.settings({ glimmer_post_menu_mode: "enabled", assign_enabled: true });
+  needs.settings({ assign_enabled: true });
 
   needs.pretender((server, helper) => {
     server.get("/assign/suggestions", () => {
@@ -121,7 +121,6 @@ acceptance("Discourse Assign | Assign Status enabled", function (needs) {
     can_assign: true,
   });
   needs.settings({
-    glimmer_post_menu_mode: "enabled",
     assign_enabled: true,
     enable_assign_status: true,
     assign_statuses: "New|In Progress|Done",
@@ -184,7 +183,6 @@ acceptance("Discourse Assign | Assign Status disabled", function (needs) {
     can_assign: true,
   });
   needs.settings({
-    glimmer_post_menu_mode: "enabled",
     assign_enabled: true,
     enable_assign_status: false,
   });
@@ -245,7 +243,6 @@ const remindersFrequency = [
 acceptance("Discourse Assign | User preferences", function (needs) {
   needs.user({ can_assign: true, reminders_frequency: remindersFrequency });
   needs.settings({
-    glimmer_post_menu_mode: "enabled",
     assign_enabled: true,
     remind_assigns_frequency: 43200,
   });
@@ -292,7 +289,6 @@ acceptance(
   function (needs) {
     needs.user({ can_assign: true, reminders_frequency: remindersFrequency });
     needs.settings({
-      glimmer_post_menu_mode: "enabled",
       assign_enabled: true,
       remind_assigns_frequency: 43200,
     });