Update PATCH /projects/:projectId/folders/:folderId endpoint to allow moving folder (no-changelog)

[RicardoE105]

Based on PR #13548

Chain of upstream PRs as of 2025-02-27

• PR feat(core): Add endpoint GET /projects/:projectId/folders (no-changelog) #13519:
  master ← ado-3270-be-add-endpoint-to-list-folders

  • PR feat(core): Add subFolderCount to GET /workflows and /folders (no-changelog) #13548:
    ado-3270-be-add-endpoint-to-list-folders ← ado-3281-be-update-endpoint-that-return-workflows-and-folders-to-also

    • PR Update PATCH /projects/:projectId/folders/:folderId endpoint to allow moving folder (no-changelog) #13574 (THIS ONE):
      ado-3281-be-update-endpoint-that-return-workflows-and-folders-to-also ← ado-3166-be-add-endpoint-to-move-folders

Summary

1 - Update endpoint to allow moving folder to another parent.
2 - Update delete endpoint to get parameters via query string instead of body. See discussions here.
3 - Update delete endpoint to account for case when the user is trying to transfer content to the same folder it's deleting.

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/ADO-3166/[be]-add-endpoint-to-move-folders

Review / Merge checklist

• PR title and summary are descriptive. (conventions)
• Docs updated or follow-up ticket created.
• Tests included.
• PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

[codecov]

Codecov Report

Attention: Patch coverage is 95.23810% with 1 line in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...n8n/api-types/src/dto/folders/delete-folder.dto.ts	0.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[Cadiac]

Nice, good catch on preventing the issue on moving workflows to the same folder that's being deleted