Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix DB migration #1066

Merged
merged 96 commits into from
Dec 6, 2024
Merged

Fix DB migration #1066

merged 96 commits into from
Dec 6, 2024

Conversation

MinhxNguyen7
Copy link
Member

Actually only parse RDS env schema when running migration.

Douglas-Hong and others added 30 commits October 24, 2024 17:25
@Douglas-Hong @MinhxNguyen7
Scan dynamo for all ids
cf5bc89
@ap0nia @MinhxNguyen7
feat(backend): db drizzle schema
49268b2
@ap0nia @MinhxNguyen7
fix(backend): db schema entrypoint
f92b827
@Douglas-Hong @MinhxNguyen7
Finish local migration
e196420
@MinhxNguyen7
chore(ddb.ts): remove unused import
49d723d
@MinhxNguyen7
fix(auth): table
60ac1d5
@MinhxNguyen7
feat(subscription): target status enum
a572caa
@MinhxNguyen7
chore(schema): rename tables to plural nouns
da1facb
@MinhxNguyen7
feat(schema): move active schedule into users table
dafd70b
@MinhxNguyen7
refactor(schema): schedule dir
1117b0c
@MinhxNguyen7
feat(schema): export some types
5464871
@MinhxNguyen7
feat(schema): remove name from user
46c359c
@MinhxNguyen7
feat(migration): proper batching and filtering in getting all data
2a013df
@MinhxNguyen7
feat(schema): clearer account types
a68b906
@MinhxNguyen7
feat(migration): properly copy users
f3caa71
@MinhxNguyen7
feat(coursesInSchedule): not null columns
897e647
@MinhxNguyen7
fix(ddb): typing for getAddUserDataBatches
a1616c7
@MinhxNguyen7
feat(user): re-add name
db8abd0
@MinhxNguyen7
feat(migrate): migration script
a5f17d4
@MinhxNguyen7
fix(migrate): return statement positioning
8596734
@MinhxNguyen7
feat(migrate) error handling
c02b374
@MinhxNguyen7
chore(trpc): remove unused endpoint
d41e2b0
@MinhxNguyen7
feat(schema): add lastUpdated to tables
d94c614
@MinhxNguyen7
feat(schema): add lastUpdated to tables
47562f3
@MinhxNguyen7
feat(schema): user's schedules must be unique
50f454f
@MinhxNguyen7
feat(api): drizzle db wrapper
9e317a3
@MinhxNguyen7
refactor(migrate): use RDS wrapper
fd9d6d8
@MinhxNguyen7
feat(api): write to both RDS and Dynamo
a764216
@MinhxNguyen7
chore: remove unused imports
d65892d
@MinhxNguyen7
feat(api): mangle duplicate schedule names
02fbb46
MinhxNguyen7 and others added 22 commits November 24, 2024 23:00
@MinhxNguyen7
fix(db): guest user account insert
74adda3
@MinhxNguyen7
feat(migration): time profiling
6d4e7bb
@MinhxNguyen7
fix(api): drop all schedules before inserting
14cc77b
@MinhxNguyen7
feat(schema): set current schedule to null on delete
591a9e0
@MinhxNguyen7
Merge remote-tracking branch 'origin/main' into migrate-rds
88956ae
@MinhxNguyen7
feat(cdk): env join from backend
536d839
@ecxyzzy
ci: hopefully fix deploy?
7076e2f
@ecxyzzy
ci: actually fix things???
4e182c6
@ecxyzzy
fljdsahfljdsahfs
d52aa25
@MinhxNguyen7
feat(dev): dotenv for backend
b7dab83
@MinhxNguyen7
fix(users router): typo
84b9e32
@ecxyzzy
feat: parallelize ddb/rds write
73113ed
@MinhxNguyen7
fix(ci): use default environment
282d847
@MinhxNguyen7
feat(saveUserData): only throw if DDB fails to write
14bc098
@MinhxNguyen7
test(ci): (temporarily) run migration
d7ecd1f
@MinhxNguyen7
Revert "fix(ci): use default environment"
b493738 
This reverts commit 282d847.
@MinhxNguyen7
feat(backend): separate env schema for RDS
e42fd7f
@MinhxNguyen7
Revert "test(ci): (temporarily) run migration"
ed86824 
This reverts commit d7ecd1f.
@MinhxNguyen7
fix(backend): env schema intersection syntax
538515c
@MinhxNguyen7
feat(ddb): more descriptive error when tableName is missing
50b6ec0
@MinhxNguyen7
fix(db): actually only use RDS env for drizzle client
b8f52d7
@MinhxNguyen7
Merge branch 'main' into migrate-rds
245be6f
@MinhxNguyen7 MinhxNguyen7 marked this pull request as ready for review December 6, 2024 07:14
@MinhxNguyen7 MinhxNguyen7 self-assigned this Dec 6, 2024
@MinhxNguyen7 MinhxNguyen7 merged commit 0f57bd6 into main Dec 6, 2024
3 of 5 checks passed
@MinhxNguyen7 MinhxNguyen7 mentioned this pull request Dec 6, 2024
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@MinhxNguyen7 MinhxNguyen7

Labels
backend bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@MinhxNguyen7 @Douglas-Hong @ap0nia @ecxyzzy @IsaacNguyen