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.

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

구현 - 박스 오피스 순위 API 및 모델 추가 #13

Merged
merged 10 commits into from
Sep 4, 2024

Conversation

ag502
Copy link
Collaborator

@ag502 ag502 commented Aug 31, 2024

PR 타입

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 문서 수정
  • 기타

연관 이슈

resolve #12

개요

  • 박스 오피스 순위 API 요청 코드 추가 및 모델 추가

변경 사항

  • Slot 컴포넌트 추가
  • Aspect-Ratio 컴포넌트 추가

리뷰시 참고할 사항

@ag502 ag502 added the feature label Aug 31, 2024
@ag502 ag502 requested a review from f-lab-james August 31, 2024 02:33
@ag502 ag502 self-assigned this Aug 31, 2024
@ag502 ag502 linked an issue Aug 31, 2024 that may be closed by this pull request
3 tasks
Copy link

...axiosConfig,
});

return { data: camelCaseObjMapper(data), ...rest };

Choose a reason for hiding this comment

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

저라면 그냥 이렇게 한 번에 처리할 것 같네요

return tmdbHttp.get('').then((res) => camelCaseObjMapper(res.data))

import { PopularMovieListRes } from "./response-types";

export default class MovieListApi {
static baseURL = "movie";

Choose a reason for hiding this comment

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

이건 외부에 노출시킬 필요도 없으니 8번째 라인쯤에서 관리해도 되겠네요

"@vanilla-extract/dynamic": "*",
"@vanilla-extract/next-plugin": "*",
"@vanilla-extract/recipes": "*",
"@vanilla-extract/sprinkles": "*",

Choose a reason for hiding this comment

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

pnpm -r update @vanilla-extract/css@"^1.15.3"

@ag502 ag502 merged commit e523cd2 into feature/10-api-communication Sep 4, 2024
2 checks passed
@ag502 ag502 deleted the feature/12-popular-movies-api branch September 4, 2024 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] 박스 오피스 순위 API 및 모델 추가
2 participants