Skip to content
This repository has been archived by the owner on Sep 17, 2024. It is now read-only.

feat: Create uploads module #76

Merged
merged 1 commit into from
Jun 10, 2024
Merged

feat: Create uploads module #76

merged 1 commit into from
Jun 10, 2024

Conversation

Blckbrry-Pi
Copy link
Collaborator

@Blckbrry-Pi Blckbrry-Pi commented Mar 30, 2024 •
edited
Loading

Resolves OGB-19

@Blckbrry-Pi Graphite App
Copy link
Collaborator Author

Blckbrry-Pi commented Mar 30, 2024 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Blckbrry-Pi and the rest of your teammates on Graphite Graphite

@Blckbrry-Pi Blckbrry-Pi force-pushed the 03-29-feat_create_uploads_module branch 2 times, most recently from 54f9928 to af9002f Compare March 31, 2024 22:54
@Blckbrry-Pi Blckbrry-Pi marked this pull request as ready for review March 31, 2024 23:09
@linear Linear
Copy link

linear bot commented Mar 31, 2024

OGB-19 `uploads` module

@Blckbrry-Pi Blckbrry-Pi force-pushed the 03-29-feat_create_uploads_module branch from af9002f to bd5c746 Compare April 1, 2024 00:41
@Blckbrry-Pi Blckbrry-Pi mentioned this pull request Apr 1, 2024
Open
@Blckbrry-Pi Blckbrry-Pi mentioned this pull request Apr 1, 2024
Closed
MasterPtato
MasterPtato approved these changes Apr 1, 2024
View reviewed changes
modules/uploads/scripts/list_for_user.ts Outdated Show resolved Hide resolved
@Blckbrry-Pi Blckbrry-Pi force-pushed the 03-29-feat_create_uploads_module branch 2 times, most recently from bc110e5 to 1d39eaf Compare April 1, 2024 18:19
NathanFlurry
NathanFlurry suggested changes Apr 16, 2024
View reviewed changes
Copy link
Member

@NathanFlurry NathanFlurry left a comment

Choose a reason for hiding this comment

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

Nice work. Big change is the chunked files.

modules/uploads/scripts/prepare.ts Outdated Show resolved Hide resolved
modules/uploads/scripts/prepare.ts Outdated Show resolved Hide resolved
modules/uploads/scripts/prepare.ts Outdated Show resolved Hide resolved
modules/uploads/db/schema.prisma Outdated Show resolved Hide resolved
modules/uploads/db/schema.prisma Outdated Show resolved Hide resolved
modules/uploads/scripts/complete.ts Outdated Show resolved Hide resolved
modules/uploads/config.ts Outdated Show resolved Hide resolved
modules/uploads/scripts/prepare.ts Show resolved Hide resolved
modules/uploads/utils/bucket.ts Outdated Show resolved Hide resolved
modules/uploads/scripts/get.ts Show resolved Hide resolved
NathanFlurry
NathanFlurry suggested changes Apr 16, 2024
View reviewed changes
modules/uploads/config.ts Outdated Show resolved Hide resolved
@Blckbrry-Pi Blckbrry-Pi force-pushed the 03-29-feat_create_uploads_module branch 3 times, most recently from c26e522 to b751ee9 Compare April 17, 2024 16:23
@Blckbrry-Pi Blckbrry-Pi force-pushed the 03-29-feat_create_uploads_module branch from b751ee9 to 9366389 Compare April 17, 2024 17:03
NathanFlurry
NathanFlurry suggested changes Apr 17, 2024
View reviewed changes
modules/uploads/config.ts Outdated Show resolved Hide resolved
...oads/db/migrations/20240417160058_add_multipart_i_ds_to_multipart_file_uploads/migration.sql Outdated Show resolved Hide resolved
modules/uploads/db/schema.prisma Show resolved Hide resolved
modules/uploads/scripts/prepare.ts Outdated Show resolved Hide resolved
modules/uploads/scripts/prepare.ts Outdated Show resolved Hide resolved
modules/uploads/scripts/get.ts Show resolved Hide resolved
modules/uploads/scripts/get.ts Outdated Show resolved Hide resolved
modules/uploads/scripts/get.ts Outdated Show resolved Hide resolved
modules/uploads/scripts/list_for_user.ts Outdated Show resolved Hide resolved
modules/uploads/scripts/list_for_user.ts Outdated Show resolved Hide resolved
@Blckbrry-Pi Blckbrry-Pi force-pushed the 03-29-feat_create_uploads_module branch from 9366389 to 2efca85 Compare April 19, 2024 13:44
NathanFlurry
NathanFlurry suggested changes Apr 22, 2024
View reviewed changes
Copy link
Member

@NathanFlurry NathanFlurry left a comment

Choose a reason for hiding this comment

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

nice, this is very close. this is going to be super super useful once it's merged.

modules/uploads/db/schema.prisma Show resolved Hide resolved
modules/uploads/scripts/complete.ts Outdated Show resolved Hide resolved
modules/uploads/utils/types.ts Outdated Show resolved Hide resolved
modules/uploads/scripts/delete.ts Show resolved Hide resolved
modules/uploads/utils/env.ts Outdated Show resolved Hide resolved
modules/uploads/utils/env.ts Outdated Show resolved Hide resolved
@Blckbrry-Pi Blckbrry-Pi force-pushed the 03-29-feat_create_uploads_module branch 2 times, most recently from 5cb668f to 10c688f Compare April 23, 2024 11:59
@Blckbrry-Pi Blckbrry-Pi mentioned this pull request Apr 24, 2024
Open
MasterPtato
MasterPtato suggested changes Apr 24, 2024
View reviewed changes
modules/uploads/utils/bucket.ts Outdated Show resolved Hide resolved
modules/uploads/utils/types.ts Outdated Show resolved Hide resolved
modules/uploads/utils/types.ts Outdated Show resolved Hide resolved
modules/uploads/utils/bucket.ts Show resolved Hide resolved
@Blckbrry-Pi Blckbrry-Pi changed the base branch from 05-18-chore_update_imports_for_module.gen.ts to main May 21, 2024 18:41
@Blckbrry-Pi Blckbrry-Pi force-pushed the 03-29-feat_create_uploads_module branch 3 times, most recently from 9e8c3b8 to d9f0c5b Compare May 22, 2024 00:01
@Blckbrry-Pi Blckbrry-Pi force-pushed the 03-29-feat_create_uploads_module branch from d9f0c5b to 3c40379 Compare May 29, 2024 17:21
NathanFlurry
NathanFlurry suggested changes Jun 4, 2024
View reviewed changes
modules/uploads/db/schema.prisma Outdated Show resolved Hide resolved
modules/uploads/scripts/list_by_tags.ts Outdated Show resolved Hide resolved
modules/uploads/db/schema.prisma Show resolved Hide resolved
modules/uploads/db/migrations/20240521193107_userid_to_tag/migration.sql Outdated Show resolved Hide resolved
NathanFlurry
NathanFlurry suggested changes Jun 4, 2024
View reviewed changes
modules/uploads/scripts/prepare.ts Outdated Show resolved Hide resolved
NathanFlurry
NathanFlurry previously requested changes Jun 4, 2024
View reviewed changes
modules/uploads/db/schema.prisma Outdated Show resolved Hide resolved
@NathanFlurry NathanFlurry force-pushed the 03-29-feat_create_uploads_module branch from 3c40379 to cb8c42d Compare June 7, 2024 23:38
@NathanFlurry NathanFlurry changed the base branch from main to 06-07-chore_migrate_from_yaml_to_json June 7, 2024 23:38
This was referenced Jun 7, 2024
Open
Closed
Merged
Merged
@NathanFlurry NathanFlurry force-pushed the 03-29-feat_create_uploads_module branch 3 times, most recently from 1db2dcd to 2c11c57 Compare June 10, 2024 04:13
@NathanFlurry NathanFlurry changed the base branch from 06-07-chore_migrate_from_yaml_to_json to 06-08-chore_re-enable_rate_limit_module June 10, 2024 04:13
@NathanFlurry NathanFlurry mentioned this pull request Jun 10, 2024
Merged
@NathanFlurry NathanFlurry force-pushed the 03-29-feat_create_uploads_module branch 2 times, most recently from 875a790 to 8cb913b Compare June 10, 2024 04:38
@NathanFlurry NathanFlurry changed the base branch from 06-08-chore_re-enable_rate_limit_module to main June 10, 2024 04:38
@NathanFlurry NathanFlurry force-pushed the 03-29-feat_create_uploads_module branch 2 times, most recently from 8173f65 to 34ef6be Compare June 10, 2024 05:33
@NathanFlurry NathanFlurry force-pushed the 03-29-feat_create_uploads_module branch from 34ef6be to b2312b2 Compare June 10, 2024 05:34
@NathanFlurry NathanFlurry merged commit 442cbb3 into main Jun 10, 2024
2 checks passed
@github-actions github-actions bot mentioned this pull request Jun 10, 2024
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@MasterPtato MasterPtato MasterPtato approved these changes

@NathanFlurry NathanFlurry NathanFlurry left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Blckbrry-Pi @NathanFlurry @MasterPtato