Merge branch 'remove-group-full-path-param-for-ordering' into 'master'

Remove group_full_path param when ordering issues

See merge request gitlab-org/gitlab!80180

app/assets/javascripts/issues/list/components/issues_list_app.vue

@@ -544,7 +544,6 @@ export default {
         .put(joinPaths(issueToMove.webPath, 'reorder'), {
           move_before_id: isMovingToBeginning ? null : getIdFromGraphQLId(moveBeforeId),
           move_after_id: isMovingToEnd ? null : getIdFromGraphQLId(moveAfterId),
-          group_full_path: this.isProject ? undefined : this.fullPath,
         })
         .then(() => {
           const serializedVariables = JSON.stringify(this.queryVariables);

app/assets/javascripts/issues/manual_ordering.js

@@ -7,12 +7,11 @@ import createFlash from '~/flash';
 import axios from '~/lib/utils/axios_utils';
 import { s__ } from '~/locale';
 
-const updateIssue = (url, issueList, { move_before_id, move_after_id }) =>
+const updateIssue = (url, { move_before_id, move_after_id }) =>
   axios
     .put(`${url}/reorder`, {
       move_before_id,
       move_after_id,
-      group_full_path: issueList.dataset.groupFullPath,
     })
     .catch(() => {
       createFlash({
@@ -52,7 +51,7 @@ const initManualOrdering = () => {
         const beforeId = prev && parseInt(prev.dataset.id, 10);
         const afterId = next && parseInt(next.dataset.id, 10);
 
-        updateIssue(url, issueList, { move_after_id: afterId, move_before_id: beforeId });
+        updateIssue(url, { move_after_id: afterId, move_before_id: beforeId });
       },
     }),
   );

app/views/shared/_issues.html.haml

 = render 'shared/alerts/positioning_disabled' if @sort == 'relative_position'
 
 - if @issues.to_a.any?
-  %ul.content-list.issues-list.issuable-list{ class: issue_manual_ordering_class, data: { group_full_path: @group&.full_path } }
+  %ul.content-list.issues-list.issuable-list{ class: issue_manual_ordering_class }
     = render partial: 'projects/issues/issue', collection: @issues
   = paginate @issues, theme: "gitlab"
 - else

spec/frontend/issues/list/components/issues_list_app_spec.js

@@ -735,7 +735,6 @@ describe('CE IssuesListApp component', () => {
                   data: JSON.stringify({
                     move_before_id: getIdFromGraphQLId(moveBeforeId),
                     move_after_id: getIdFromGraphQLId(moveAfterId),
-                    group_full_path: isProject ? undefined : defaultProvide.fullPath,
                   }),
                 });
               });