Skip to content

Conversation

@Rakemoon
Copy link

Week 4. still in progress.

Rakemoon and others added 13 commits June 15, 2024 18:10
still in progress...
Ada beberapa eslint rule yang saya matikan, karena berbentrokan dengan
typescript :
- import/unresolved
- import/extensions
wkkwk hmpir lupa sama coursenya
- AddMiddlewares decortaor register humoungous number of middlewares
- Dfs walk trying to import non javascript files
- remove herobrine
Copy link
Contributor

Choose a reason for hiding this comment

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

ini lu bikin router versi sendiri ?

Copy link
Contributor

Choose a reason for hiding this comment

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

ini jugaa, buat apa fungsinya

Copy link
Contributor

@zexoverz zexoverz left a comment

Choose a reason for hiding this comment

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

ini semacam lu nambahin sistem register router ya, biar kalo ada router baru tinggal di regist aja.

Comment on lines 4 to 14
import UserService from '../../../../services/user';
import ApiError from '../../../../lib/utils/ApiError';
import TokenService from '../../../../services/generateToken';
import { AddMiddleware } from '../../../../lib/utils/RouterDecorator';
import ValidationMiddleware from '../../../../lib/middlewares/ValidationMiddleware';
import AuthValidation from '../../../../lib/validations/AuthValidations';

@AddMiddleware(ValidationMiddleware.validate(AuthValidation.register))
export default class extends RouterBuilder {
public static override async controller(req: Request, res: Response) {
const existing = await UserService.getEmail(req.body.email);
Copy link
Contributor

Choose a reason for hiding this comment

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

oke pas lihat ini paham gua, cuman bedain cara ngebuat route nya aja ya.

Copy link
Contributor

@zexoverz zexoverz left a comment

Choose a reason for hiding this comment

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

Gua approve walaupun beda struktur, selama lu paham pake sturktur lu.
oiya tambahin readme.md dan kasih alasan kenapa pake RouterBuilder, biar yang lain juga bisa belajar

Rakemoon added 2 commits June 19, 2024 11:36
- Menyelesaikan Semua CRUD
- System Quantity stock
- Api search menggunakan category /v1/products/search
- Pagination untuk semua getAll api
- Hanya admin yang dapat mengakses /v1/users
Copy link
Contributor

@zexoverz zexoverz left a comment

Choose a reason for hiding this comment

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

Nice document, lanjutkan

@Rakemoon
Copy link
Author

mohon reviewnya sir, test suite dan dokumentasi sudah saya kerjakan

Copy link
Contributor

@zexoverz zexoverz left a comment

Choose a reason for hiding this comment

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

Dokumentasi Approved, sorry gua baru review

@zexoverz
Copy link
Contributor

Congrats dzikrifauzi, selamat atas kelulusannya di phase 1 🔥 🔥 🔥
lu termasuk devil, soalnya bantai 5 week tepat waktu. Progress belajar terlalu cepat wkkww, keknya karna lu udah berpengalaman ya di programming. Final Project juga perfect, termasuk top project selama group project phase 1.

overall nilai lu di phase 1 ada di angka 98% (hampir sempurna), penilaian ini base on:
-Logic Handling
-Explore
-Question Ask
-Learn Speed
-Task Submit

Review:
Karena lu devil, review gua ga ada sih. overall lu udah paham fundamental backend dari awal, paling dari gua banyakin jam terbang aja sih dan buat portofolio yang complex di phase 2 nanti. Mumpung logic lu lagi kuat kuat nya nih, di manfaatin buat project" yang bagus biar nyicil buat profile career lu nanti.

untuk open gate phase 2 nanti di umumin jadwalnya, standby aja.

bisa explore backend lagi kalau tertarik mau jadi backend developer :
-Explore Payment Gateway (Stripe)
-Explore Cache (redis)
-Explore Microservice Architecture

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.

2 participants