ENG-2218 Copy one asset from Supabase into Roam storage - #1397
Open
maparent wants to merge 1 commit into
Conversation
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 6, 2026 14:37
f597a47 to
51a1453
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 6, 2026 15:04
51a1453 to
0c4df72
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
2 times, most recently
from
September 6, 2026 16:40
06de101 to
8653cb0
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 6, 2026 18:09
8653cb0 to
23e7946
Compare
maparent
marked this pull request as ready for review
September 6, 2026 20:09
Contributor
PR size/scope checkThis PR is over our review-size guideline.
Please split this into smaller PRs unless there is a clear reason the changes need to land together. If keeping it as one PR, please add a brief justification covering:
|
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 6, 2026 22:32
23e7946 to
bbe9674
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 6, 2026 22:41
bbe9674 to
0e1b241
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 6, 2026 22:59
0e1b241 to
f88260d
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 8, 2026 15:35
f88260d to
7fc2d70
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 8, 2026 15:40
7fc2d70 to
42239eb
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 8, 2026 22:06
7bbe653 to
f79a205
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 9, 2026 13:26
f79a205 to
eda3218
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 9, 2026 15:58
eda3218 to
73b6756
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 9, 2026 16:10
73b6756 to
7c0dc7b
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 9, 2026 16:28
7c0dc7b to
6210109
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 9, 2026 18:24
6210109 to
6ee1905
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 9, 2026 18:37
6ee1905 to
53d5a0c
Compare
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 9, 2026 18:42
53d5a0c to
8b0cd0d
Compare
maparent
removed this pull request from stack #1393
September 10, 2026 14:42
maparent
added this pull request to stack #1430
September 10, 2026 14:43
maparent
force-pushed
the
eng-2218-Copy-assets-from-supabase-into-Roam-storage
branch
from
September 10, 2026 20:45
8b0cd0d to
1c8c7d1
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reviewer brief
This is introducing a function that copies assets from firebase to supabase.
It is exercised in following PRs.
Verification
Unit tests.
Loom video
https://www.loom.com/share/244e8b7981434353b2407984c902a0f8
Scope check
$scope-checkagainst the ENG ticket and final diff.Local delegated full review
$dg-delegated-full-reviewwhen no other full-review workflow is available.https://linear.app/discourse-graphs/issue/ENG-2218/copy-one-asset-from-supabase-into-roam-storage