search

cloud-barista / cm-data-mold

Apache License 2.0
2 stars 1 forks source link

refactor : replace Resp interface to models #38

Closed zcon-cychoi closed 10 months ago

zcon-cychoi commented 10 months ago
  1. models 패키지 추가
  2. Response map[string]interface{} 를 models 의 struct 타입으로 변경
  3. 마이그레이션 관련 페이지 핸들러들을 controllers 의 pageHandlers.go 로 이동
yunkon-kim commented 10 months ago

@zcon-cychoi

아래 사항은 별도로 진행중이실까요? PR 타이틀 상으로 Request Body를 포함하는 것 같아 문의드리는 부분입니다 ^^

(Ref. #35 코멘트)

@zcon-cychoi 넵 설명 감사드립니다. 자세히 설명해 주셔서 바로바로 이해가 잘 되네요 😃

Migration 이외의 파트를 제가 구분하여 PR 올리도록 하겠습니다.

Migration 관련 파트는 Credential 관련 부분을 분리해야할 필요가 있고요. 아래 두 가지 안 정도를 생각해 볼 수 있을 것 같습니다. 1안) cm-data-mold binary와 같은 디렉토리에 credential을 사전 세팅 2안) Credential을 최초 한번 업로드하고, 이후 Migration시 file의 ID 또는 path를 활용하는 방안

관련하여 수정 가능여부, 공수, 이슈 등을 먼저 파악해 보려고 합니다. 확인해 보시고 공유해주시기 바랍니다. (코드 개선은 이후에 진행하도록 하겠습니다.)

zcon-cychoi commented 10 months ago

@yunkon-kim request 부분은 수정하지 않았네요. 혼선을 드려 죄송합니다. crendential 부분은 따로 진행해보겠습니다

yunkon-kim commented 10 months ago

@zcon-cychoi 넵! 현황 및 계획을 공유해주셔서 감사드립니다!

그럼 이 PR에서는 리뷰 코멘트 사항에 대해서만 다루도록 하겠습니다 ^^