-
Notifications
You must be signed in to change notification settings - Fork 16
Wei #1
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
base: main
Are you sure you want to change the base?
Wei #1
Conversation
|
good job wei, bagus juga explore GraphQL nya. lanjut ke inventory system ya, push nya ke repo ini |
|
Terimah kasih banyak kak. Saya sudah mengerjakan tugas terkait Inventory System kak.
Jujur saya tidak tau, mengapa saat saya sudah menghapus token nya dari database, user masih dapat mengakses. dimana pada crud: POST, PUT, DELETE: |
|
Overall crud nya udah OK , Quantity System nya juga rapih. order Item update dan delete juga di handle effect totalPricsnya. untuk getProduct by user atau getOrder by user harusnya bisa pake include nya prisma aja biar querynya lebih cepet, ketimbang nyari pake where. buat pagination kalau bisa nama variablenya jangan skip dan take, soalnya kadang frontend ga tau itu apa. bagus dibikin page dan size (page mengacu kepada page berapa yang mau di get, dan size berapa total data di setiap page) untuk fitur logout caranya bukan nge delete tokennya, karena pada dasarnya JWT token itu kalau sudah generate dia bakal terus ada hashing codenya di dalam token. makanya selama hashing itu dipake buat auth , ya auth nya tetep jalan karena payloadnya masih ada didalam hash itu. coba diliat di schema Token , sebenernya kita punya kolom namanya blacklisted. blacklisted ini sebenernya fungsi untuk membedakan mana token aktif atau engga. dari pada nge delete tokennya , bagus kita mainkan status blacklisted ini agar history token tetap ada di database (jadi kita tau kapan user login dan logout) kalau mau develop fitur logout, seperti ini :
jadi logout disini methodnya PUT/PATCH karena dia hanya update status blacklisted Good job wei, lanjut live class rabu malem ya. |
|
Selamat Sore Kak Mohon di review ya untuk integration API TESTING serta Dokumentasi nya Maaf jika saya tidak bisa buat sebaik Mungkin, mau dari API TESTING atau Dokumentasi, karena ini pertama kalinya saya untuk buat Dokumentasi yang cukup kompleks dan API TESTING. Sama saya ingin mention API TESTING untuk route /api/order-items. Terimah Kasih Kak. |
zexoverz
left a comment
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.
Okey nice try wei, yang penting udah berusaha push testingnya. nanti kalo ada waktu kita bole bahas problem testing order-item ini.
lanjut group project dlu ya
|
Hello wei, selamat atas kelulusannya di phase 1 🔥 🔥 🔥
Review: Minggu besok libur ya 1 week, setelah libur nanti kita meet bareng untuk diskusi terkait mulainya phase 2. kita diskusi antara nunggu yang snowball lulus atau kita langsung gas phase 2 ber 4. Untuk mengisi liburan , bisa explore backend lagi kalau tertarik mau jadi backend developer.
|
Apa saja yang telah saya lakukan: