Skip to content

Create Firebase File model and migration file#113

Merged
gavxue merged 3 commits intomainfrom
INTW26-create-firebase-file-model
Feb 11, 2026
Merged

Create Firebase File model and migration file#113
gavxue merged 3 commits intomainfrom
INTW26-create-firebase-file-model

Conversation

@gavxue
Copy link
Collaborator

@gavxue gavxue commented Feb 8, 2026

Notion ticket link

Create Firebase File Model + Migration File

Implementation description

  • added FirebaseFile types
  • created File sequelize model
  • created migration file to generate new sql table

Steps to test

  1. run these commands in backend/typescript
docker-compose exec ts-backend bash
node migrate up
  1. verify that the files table is created with all appropriate columns
  2. test rollback with node migrate down

What should reviewers focus on?

  • model makes sense
  • follows codebase patterns

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@gavxue gavxue self-assigned this Feb 8, 2026
@gavxue gavxue marked this pull request as ready for review February 8, 2026 00:39
@gavxue gavxue requested a review from mxc-maggiechen February 8, 2026 00:39
Copy link
Collaborator

@SaqAsh SaqAsh left a comment

Choose a reason for hiding this comment

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

LGTM

@gavxue gavxue merged commit e684b4a into main Feb 11, 2026
1 check passed
@gavxue gavxue deleted the INTW26-create-firebase-file-model branch February 11, 2026 00:44
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.

3 participants