Skip to content

Add endpoints for deleting content to supported APIs for Archive#895

Open
psatala-box wants to merge 1 commit intomainfrom
ddoc-1455/add-endpoints-to-archive-apis-guide
Open

Add endpoints for deleting content to supported APIs for Archive#895
psatala-box wants to merge 1 commit intomainfrom
ddoc-1455/add-endpoints-to-archive-apis-guide

Conversation

@psatala-box
Copy link
Collaborator

Description

Added endpoints to list of supported APIs for Box Archive. The new endpoints support deletion of files and folders.

Fixes # (issue)
Re DDOC-1455

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own changes
  • I have run yarn lint to make sure my changes pass all linters
  • I have pulled the latest changes from the upstream developer branch

Contribution guidelines

For contribution guidelines, styleguide, and other helpful information please
see the CONTRIBUTING.md file in the root of this project.

Comment on lines +41 to +42
| [`DELETE /files/:id`][Delete file] | Permanently delete file within an archive. |
| [`DELETE /folders/:id`][Delete folder] | Permanently delete folder within an archive. Requires contacting customer support to enable. |
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only enabling folder deletion requires contacting customer. This is due to legacy reasons

Copy link
Collaborator

@lszal lszal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just two minor suggestions.

| [`GET /folders/:id/items`][List folder items] | List items within an archive or items in a folder within an archive. |
| [`POST /files/:id/copy`][Copy file] | Copy a file to an archive. |
| [`POST /folders/:id/copy`][Copy folder] | Copy a folder to an archive. |
| [`DELETE /files/:id`][Delete file] | Permanently delete file within an archive. |
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
| [`DELETE /files/:id`][Delete file] | Permanently delete file within an archive. |
| [`DELETE /files/:id`][Delete file] | Permanently delete a file within an archive. |

| [`POST /files/:id/copy`][Copy file] | Copy a file to an archive. |
| [`POST /folders/:id/copy`][Copy folder] | Copy a folder to an archive. |
| [`DELETE /files/:id`][Delete file] | Permanently delete file within an archive. |
| [`DELETE /folders/:id`][Delete folder] | Permanently delete folder within an archive. Requires contacting customer support to enable. |
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
| [`DELETE /folders/:id`][Delete folder] | Permanently delete folder within an archive. Requires contacting customer support to enable. |
| [`DELETE /folders/:id`][Delete folder] | Permanently delete a folder within an archive. Requires contacting customer support to enable. |

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants