Skip to content

feat(cloudinary): expose the upload's public id and allow deleting an upload [Deel sync 3/5] - #4773

Open
gilgardosh wants to merge 1 commit into
claude/deel-sync-2-settings-uifrom
claude/deel-sync-3-cloudinary-delete
Open

gilgardosh wants to merge 1 commit into
claude/deel-sync-2-settings-uifrom
claude/deel-sync-3-cloudinary-delete

Conversation

@gilgardosh

Copy link
Copy Markdown
Collaborator

Stack (Deel sync refactor), merge in order:

  1. DB migration + provider-credentials
  2. Deel business picker in settings
  3. 👉 this PR: Cloudinary upload delete
  4. Deel API client hardening
  5. Atomic Deel sync (feat(deel): sync Deel documents atomically per receipt, with descriptive issues [Deel sync 5/5] #4768)

This change doesn't depend on PRs 1 and 2. It's stacked only to keep the merge order linear.

Summary

  • uploadInvoiceToCloudinary also returns Cloudinary's publicId and resourceType. Both fields are optional, so existing callers are unaffected.
  • New deleteUpload(publicId, resourceType), a wrapper around cloudinary.uploader.destroy.
  • PR 5 uses it to remove invoice files whose save was rolled back.

Test plan

  • Server typecheck passes. The two errors in email-ingestion-control.resolver.test.ts are also on main.
  • The documents and app-providers unit tests pass.

Generated by Claude Code

@gilgardosh
gilgardosh deployed to accounter-fullstack October 20, 2026 14:45 — with GitHub Actions Active
@gilgardosh
gilgardosh deployed to accounter-fullstack October 20, 2026 14:45 — with GitHub Actions Active
@gilgardosh gilgardosh self-assigned this Oct 20, 2026
@gilgardosh
gilgardosh added this pull request to stack #4775 October 20, 2026 15:11
@github-actions

Copy link
Copy Markdown
Contributor

The latest changes of this PR are not available as alpha, since there are no linked changesets for this PR.

… upload

Uploads now also return Cloudinary's `publicId` and `resourceType` (optional fields; existing
callers are unaffected), and `deleteUpload(publicId, resourceType)` removes a file — used next by
the Deel sync to remove files of invoices whose save was rolled back.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015CMcZduVAnhW3xJAtvmiQU
@gilgardosh
gilgardosh force-pushed the claude/deel-sync-3-cloudinary-delete branch from 6647918 to 3449b82 Compare October 20, 2026 16:08
@gilgardosh
gilgardosh deployed to accounter-fullstack October 20, 2026 16:08 — with GitHub Actions Active
@gilgardosh
gilgardosh deployed to accounter-fullstack October 20, 2026 16:08 — with GitHub Actions Active

This branch was successfully deployed

1 active deployment
accounter-fullstack — 3449b827 Deployed Oct 20, 2026 by gilgardosh via Hive schema check / check #9956
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.

2 participants