Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use DELETE /assets/<id> for removing Enso_File assets #9288

Closed
radeusgd opened this issue Mar 5, 2024 · 0 comments · Fixed by #9551
Closed

Use DELETE /assets/<id> for removing Enso_File assets #9288

radeusgd opened this issue Mar 5, 2024 · 0 comments · Fixed by #9551
Assignees
Labels
-libs Libraries: New libraries to be implemented l-cloud-integration Enso Cloud integration work

Comments

@radeusgd
Copy link
Member

radeusgd commented Mar 5, 2024

We now use asset-type-specific endpoints, but some of them do not work, we should refactor to use the 'generic' DeleteAsset command.

We need to set the force query parameter to true to avoid sending the elements to trash instead.

@radeusgd radeusgd self-assigned this Mar 5, 2024
@github-project-automation github-project-automation bot moved this to ❓New in Issues Board Mar 5, 2024
@radeusgd radeusgd added -libs Libraries: New libraries to be implemented l-cloud-integration Enso Cloud integration work labels Mar 5, 2024
@AdRiley AdRiley moved this from ❓New to 📤 Backlog in Issues Board Mar 6, 2024
@radeusgd radeusgd moved this from 📤 Backlog to 🔧 Implementation in Issues Board Mar 26, 2024
@radeusgd radeusgd moved this from 🔧 Implementation to 👁️ Code review in Issues Board Mar 26, 2024
@mergify mergify bot closed this as completed in #9551 Mar 26, 2024
mergify bot pushed a commit that referenced this issue Mar 26, 2024
@github-project-automation github-project-automation bot moved this from 👁️ Code review to 🟢 Accepted in Issues Board Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-libs Libraries: New libraries to be implemented l-cloud-integration Enso Cloud integration work
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant