-
Notifications
You must be signed in to change notification settings - Fork 0
file uploads timeline layers #16
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
Merged
Merged
Changes from all commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
c7cf14c
feat: file uploads, video/audio layers, enter/exit time, and AI captions
claude 17aeb61
Merge branch 'main' into claude/file-uploads-timeline-layers-XE0ue
epavanello 865cc87
bkp
epavanello 29653f3
wip
epavanello a4cd383
feat: Implement video and camera capture, add project thumbnail gener…
epavanello 4884550
feat: Introduce AlertDialog and Popover components and extend layer s…
epavanello a217fb6
fix
epavanello d06b1f5
cleanup
epavanello 4966df6
wip
epavanello 85dbc9c
ui fix
epavanello 44f21a5
keyframes refactor
epavanello 2d9bbb6
bkp
epavanello e44c85c
fix video sync
epavanello 38bad8b
video fix
epavanello d61fd37
captions! 💣
epavanello 6e815cd
secured captions
epavanello 24b4839
videos
epavanello 2a0b7f8
ux
epavanello File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| Handle this need, handle/support file upload properly for images and videos, actually image accept only src, should accept also a file. Then allow that the project store securely files on s3 like configurable storage provider. And support the preview. The video layer should define the seconds from start and and, need to be rendered in a proper way in the timeline, and let users to cut/split/resize it. Be sure that server side rendering will show the proper frames in the capture, and merge the audio tracks properly. Also allow to upload/record audios, and manage properly via vercel ai and openrouterr providers the caption generator and preview (this probably should be an audio layer with captions style to be shown ) also for audio handle properly the timeline, split, crop, move. Apply the same concept of enter/exit time for other layers, so we can handle more layers together and visually. Don’t struggle to fix errors, the environment can’t be tested or checked completely, so implement it and I’ll test and tune it. |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| ALTER TABLE "project" ADD COLUMN "thumbnail_url" text; |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| CREATE TABLE "asset" ( | ||
| "id" text PRIMARY KEY NOT NULL, | ||
| "project_id" text NOT NULL, | ||
| "user_id" text NOT NULL, | ||
| "storage_key" text NOT NULL, | ||
| "url" text NOT NULL, | ||
| "original_name" text NOT NULL, | ||
| "mime_type" text NOT NULL, | ||
| "media_type" text NOT NULL, | ||
| "size" integer NOT NULL, | ||
| "created_at" timestamp DEFAULT now() NOT NULL | ||
| ); | ||
| --> statement-breakpoint | ||
| ALTER TABLE "asset" ADD CONSTRAINT "asset_project_id_project_id_fk" FOREIGN KEY ("project_id") REFERENCES "public"."project"("id") ON DELETE cascade ON UPDATE no action;--> statement-breakpoint | ||
| ALTER TABLE "asset" ADD CONSTRAINT "asset_user_id_user_id_fk" FOREIGN KEY ("user_id") REFERENCES "public"."user"("id") ON DELETE cascade ON UPDATE no action;--> statement-breakpoint | ||
| CREATE INDEX "asset_project_id_idx" ON "asset" USING btree ("project_id");--> statement-breakpoint | ||
| CREATE INDEX "asset_user_id_idx" ON "asset" USING btree ("user_id"); |
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build args propagate secrets into the Docker build context — same concern as the Dockerfile.
If the
ARG/ENVlines are removed from the Dockerfile per the suggestion above, these buildargsentries should be removed as well. The runtimeenvironmentblock (lines 26–32) is sufficient for server-side secrets.🤖 Prompt for AI Agents