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

[관리자] 크크쇼 메인 관리 #267

Merged
merged 74 commits into from
Mar 7, 2022
Merged

[관리자] 크크쇼 메인 관리 #267

merged 74 commits into from
Mar 7, 2022

Conversation

jiyeon2
Copy link
Contributor

@jiyeon2 jiyeon2 commented Mar 6, 2022

크크쇼 메인데이터 조회 GET /admin/kkshow-main
크크쇼 메인데이터 수정 POST /admin/kkshow-main

크크쇼 메인 데이터 응답 타입

예시 데이터 형태

테스트케이스

  • 크크쇼 메인 데이터를 db에 저장할 수 있다

  • 크크쇼 메인 데이터를 db에서 조회할 수 있다

  • 하나의 테이블에서 모든 정보를 저장 및 조회하도록 구성

  • 메인 캐러셀 영역 데이터를 관리할 수 있다

  • 라이브 예고 데이터를 관리할 수 있다

  • 베스트 라이브 데이터를 관리할 수 있다

  • 베스트 방송인 데이터를 관리할 수 있다

기타

  • next 버전 "^12.1.0" -> "^12.1.1-canary.4"로 업그레이드함 (next error)

@vercel vercel bot temporarily deployed to Preview – project-lc-broadcaster March 7, 2022 05:28 Inactive
@vercel vercel bot temporarily deployed to Preview – project-lc-admin March 7, 2022 05:28 Inactive
@vercel vercel bot temporarily deployed to Preview – project-lc-kkshow March 7, 2022 05:29 Inactive
@vercel vercel bot temporarily deployed to Preview – project-lc-seller March 7, 2022 05:29 Inactive
@vercel vercel bot temporarily deployed to Preview – project-lc-broadcaster March 7, 2022 06:52 Inactive
@vercel vercel bot temporarily deployed to Preview – project-lc-admin March 7, 2022 06:52 Inactive
@vercel vercel bot temporarily deployed to Preview – project-lc-seller March 7, 2022 06:52 Inactive
@vercel vercel bot temporarily deployed to Preview – project-lc-kkshow March 7, 2022 06:52 Inactive
@jiyeon2 jiyeon2 marked this pull request as ready for review March 7, 2022 06:52
@jiyeon2 jiyeon2 requested a review from hwasurr as a code owner March 7, 2022 06:52
Copy link
Contributor

@hwasurr hwasurr left a comment

Choose a reason for hiding this comment

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

작업량이 많네요 고생많으셨습니다

libs/hooks/src/lib/queries/useKkshowMain.tsx Outdated Show resolved Hide resolved
@hwasurr
Copy link
Contributor

hwasurr commented Mar 7, 2022

처음 빌드된 이후, 데이터가 없을 것이므로, 곧바로 데이터를 집어넣어줘야할 텐데,
이 작업을 하는 동안에는 페이지가 올바르게 표현되지 않습니다

migration 선에서 기본 데이터를 설정할 필요가 있는 것 같습니다

또한, seed를 추가하면 더 좋을 것 같습니다

// 단순배너 아이템 : 이미지 url + 이미지에 연결된 링크
export interface SimpleBannerItem extends KkshowMainCarouselItemBase, Image {
type: 'simpleBanner';
linkUrl?: string;
Copy link
Contributor

@hwasurr hwasurr Mar 7, 2022

Choose a reason for hiding this comment

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

description?: string

을 추가해주실 수 있을까요? 해당 단순 배너에 대한 한줄가량의 설명 정보가
있어야 할 것 같아서요

ex) 설맞이 특가 이벤트 이미지 -> "설맞이 특가 바로가기" 와 같이 설명하는 글

스크린샷 2022-03-07 오후 4 32 21

이미지에서는 "뚱랑이 스트레치볼 추첨 증정 이벤트" 부분입니다

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 추가하겠습니다!

@vercel vercel bot temporarily deployed to Preview – project-lc-admin March 7, 2022 08:27 Inactive
@vercel vercel bot temporarily deployed to Preview – project-lc-broadcaster March 7, 2022 08:27 Inactive
@vercel vercel bot temporarily deployed to Preview – project-lc-kkshow March 7, 2022 08:27 Inactive
@vercel vercel bot temporarily deployed to Preview – project-lc-seller March 7, 2022 08:27 Inactive
@jiyeon2
Copy link
Contributor Author

jiyeon2 commented Mar 7, 2022

처음 빌드된 이후, 데이터가 없을 것이므로, 곧바로 데이터를 집어넣어줘야할 텐데, 이 작업을 하는 동안에는 페이지가 올바르게 표현되지 않습니다

migration 선에서 기본 데이터를 설정할 필요가 있는 것 같습니다

또한, seed를 추가하면 더 좋을 것 같습니다

말씀해주신대로 migration 파일에 초기데이터 insert 구문을 추가하였습니다
그리고 시드파일에도 초기데이터를 추가하였습니다!

Copy link
Contributor

@hwasurr hwasurr left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

@hwasurr hwasurr merged commit a64de4d into dev Mar 7, 2022
@jiyeon2 jiyeon2 deleted the joni-main-management branch March 8, 2022 04:58
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.

None yet

2 participants