Skip to content

Add onedrive.delete action documentation#44730

Merged
c0ffeeca7 merged 8 commits into
home-assistant:nextfrom
leodrivera:add-onedrive-delete-service-next
May 6, 2026
Merged

Add onedrive.delete action documentation#44730
c0ffeeca7 merged 8 commits into
home-assistant:nextfrom
leodrivera:add-onedrive-delete-service-next

Conversation

@leodrivera
Copy link
Copy Markdown
Contributor

@leodrivera leodrivera commented Apr 12, 2026

Proposed change

Documents the new onedrive.delete service action added in home-assistant/core#168064.

The action allows users to delete one or more files from the OneDrive app folder directly from automations. Whether the remote files go to the Recycle Bin or are permanently removed follows the existing Delete files permanently integration option.

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

Copilot AI review requested due to automatic review settings April 12, 2026 17:59
@home-assistant home-assistant Bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Apr 12, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 12, 2026

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 0367041
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/69fa844ccc2287000826093f
😎 Deploy Preview https://deploy-preview-44730--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Documents the new onedrive.delete service action for the OneDrive integration so users can delete remote files (and optionally local copies) from automations.

Changes:

  • Added documentation for the new onedrive.delete action.
  • Documented action fields (destination_path, filename, config_entry_id) and clarified Recycle Bin vs permanent deletion behavior.

Comment thread source/_integrations/onedrive.markdown
@leodrivera leodrivera requested a review from zweckj as a code owner April 12, 2026 18:41
The delete service no longer supports deleting local files. Removed the
`filename` field from the action table and updated the description accordingly.
@home-assistant home-assistant Bot added the parent-merged The parent PR has been merged already label May 4, 2026
Comment thread source/_integrations/onedrive.markdown
@home-assistant
Copy link
Copy Markdown

home-assistant Bot commented May 5, 2026

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@home-assistant home-assistant Bot marked this pull request as draft May 5, 2026 04:25
leodrivera added 3 commits May 5, 2026 20:38
…ive.delete

- Add YAML examples for single and multi-file deletion to onedrive.delete
- Note that delete removes only files, not folders created during upload
- Add allowlist_external_dirs requirement for onedrive.upload filename
- Add OneDrive folder naming restrictions disclaimer for destination_folder
@leodrivera
Copy link
Copy Markdown
Contributor Author

Also added:

  • info about allowlist_external_dirs, just like presented in
    The path part of `filename` must be an entry in the `allowlist_external_dirs` in your [`homeassistant:`](/docs/configuration/basic/) section of your `configuration.yaml` file.
  • OneDrive naming restrictions info

@leodrivera leodrivera marked this pull request as ready for review May 6, 2026 00:05
@home-assistant home-assistant Bot requested a review from c0ffeeca7 May 6, 2026 00:05
Copy link
Copy Markdown
Contributor

@c0ffeeca7 c0ffeeca7 left a comment

Choose a reason for hiding this comment

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

Thank you, @leodrivera 👍

@c0ffeeca7 c0ffeeca7 merged commit e75a3f9 into home-assistant:next May 6, 2026
7 checks passed
@home-assistant home-assistant Bot removed the parent-merged The parent PR has been merged already label May 6, 2026
@leodrivera leodrivera deleted the add-onedrive-delete-service-next branch May 6, 2026 10:23
@github-actions github-actions Bot locked and limited conversation to collaborators May 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

has-parent This PR has a parent PR in another repo next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants