Skip to content

Add TUS support to the Supabase storage driver#23958

Merged
licitdev merged 8 commits intomainfrom
tus-supabase-support
Nov 4, 2024
Merged

Add TUS support to the Supabase storage driver#23958
licitdev merged 8 commits intomainfrom
tus-supabase-support

Conversation

@ComfortablyCoding
Copy link
Member

Scope

What's changed:

  • Added TUS support for the supabase storage driver

Potential Risks / Drawbacks

  • None

Review Notes / Questions

  • None

Fixes SER-393

@changeset-bot
Copy link

changeset-bot bot commented Nov 1, 2024

🦋 Changeset detected

Latest commit: 561ccb6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@directus/storage-driver-supabase Minor
@directus/api Patch
directus Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ComfortablyCoding ComfortablyCoding marked this pull request as ready for review November 1, 2024 20:33
@ComfortablyCoding ComfortablyCoding changed the title Add TUS support to the supabase storage driver Add TUS support to the Supabase storage driver Nov 4, 2024
@licitdev licitdev enabled auto-merge (squash) November 4, 2024 22:20
@licitdev licitdev merged commit 8d07c0d into main Nov 4, 2024
@licitdev licitdev deleted the tus-supabase-support branch November 4, 2024 22:20
@github-actions github-actions bot added this to the Next Release milestone Nov 4, 2024
vwasteels pushed a commit to Agence-Indivisible/directus that referenced this pull request Dec 5, 2024
* add tus support to supabase

* add changeset

* add missing upload url

* add uploadurl within resume upload

* remove unnecessary uploadurl

* Implement custom stream source logic

* extract default chunksize to constant

* reword changeset

---------

Co-authored-by: Hannes Küttner <kuettner.hannes@gmail.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants