-
Notifications
You must be signed in to change notification settings - Fork 0
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
refactor: migrate from firebase ecosystem to postgresql #14
Conversation
Veirt
commented
Nov 23, 2024
•
edited
Loading
edited
- auth
- subject
- learner
- tutor
- tutories
- order
- reviews
- chat
8101112
to
9b7921f
Compare
…ror, and set `req.learner`
…also the push notification chat related
@Famozzy ntar buatin yang session rating |
kira kira butuh kutulis dokumentasi tentang |
createdAt: timestamp("created_at").notNull().defaultNow(), | ||
}); | ||
|
||
export const learners = pgTable("learners", { |
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.
btw, ini yang interest(of subject) gajadi kah?
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.
iya perlu hel, tadi jga aku setup postgresql nya di docker compose |
- also changed tutoryId to tutoriesId
review, kuhandle ya. Biar cepet up ke prod |
…okens, and reviews tables for performance improvement
…r handling for handling, enhance race condition handling in chat message tests
…nce tests, also handling of possibility of race condition during testing
overall udah fully migrated. Except bagian chat yang masih pake realtimedb + messaging. Any suggestion? |
Kalo dah aman merge aja |