fix: Remove trailing slashes from makers/models routes
FastAPI's redirect_slashes was causing /makers/ to redirect to /makers which then didn't match the route definition. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -123,13 +123,13 @@ def get_cars(
|
||||
|
||||
|
||||
# Makers - Must be defined before /{car_id} to avoid route conflict
|
||||
@router.get("/makers/", response_model=List[CarMakerResponse])
|
||||
@router.get("/makers", response_model=List[CarMakerResponse])
|
||||
def get_makers(db: Session = Depends(get_db)):
|
||||
"""제조사 목록 조회"""
|
||||
return db.query(CarMaker).all()
|
||||
|
||||
|
||||
@router.post("/makers/", response_model=CarMakerResponse)
|
||||
@router.post("/makers", response_model=CarMakerResponse)
|
||||
def create_maker(maker_data: CarMakerCreate, db: Session = Depends(get_db)):
|
||||
"""제조사 등록"""
|
||||
existing = db.query(CarMaker).filter(CarMaker.code == maker_data.code).first()
|
||||
@@ -144,7 +144,7 @@ def create_maker(maker_data: CarMakerCreate, db: Session = Depends(get_db)):
|
||||
|
||||
|
||||
# Models - Must be defined before /{car_id} to avoid route conflict
|
||||
@router.get("/models/", response_model=List[CarModelResponse])
|
||||
@router.get("/models", response_model=List[CarModelResponse])
|
||||
def get_models(maker_id: Optional[int] = None, db: Session = Depends(get_db)):
|
||||
"""모델 목록 조회"""
|
||||
query = db.query(CarModel)
|
||||
@@ -153,7 +153,7 @@ def get_models(maker_id: Optional[int] = None, db: Session = Depends(get_db)):
|
||||
return query.all()
|
||||
|
||||
|
||||
@router.post("/models/", response_model=CarModelResponse)
|
||||
@router.post("/models", response_model=CarModelResponse)
|
||||
def create_model(model_data: CarModelCreate, db: Session = Depends(get_db)):
|
||||
"""모델 등록"""
|
||||
existing = db.query(CarModel).filter(
|
||||
|
||||
@@ -63,13 +63,13 @@ export const carsApi = {
|
||||
},
|
||||
|
||||
getMakers: async (): Promise<CarMaker[]> => {
|
||||
const { data } = await api.get('/cars/makers/');
|
||||
const { data } = await api.get('/cars/makers');
|
||||
return data;
|
||||
},
|
||||
|
||||
getModels: async (makerId?: number): Promise<CarModel[]> => {
|
||||
const params = makerId ? { maker_id: makerId } : {};
|
||||
const { data } = await api.get('/cars/models/', { params });
|
||||
const { data } = await api.get('/cars/models', { params });
|
||||
return data;
|
||||
},
|
||||
|
||||
|
||||
Reference in New Issue
Block a user