deploy v0.1 추가 #144
deploy v0.1 추가 #144
Conversation
ISSUES CLOSED: unzipper, axios 설치
|
|
||
| controller = module.get<KoreaStockInfoController>(KoreaStockInfoController); | ||
| }); | ||
|
|
There was a problem hiding this comment.
여기는 따로 테스트 환경만 세팅해 놓은 건가요?
There was a problem hiding this comment.
nestjs-cli로 만들어 기본으로 나온 건데, 일단 삭제처리하도록 하겠습니다.
| import { Entity, Column, PrimaryGeneratedColumn } from 'typeorm'; | ||
|
|
||
| @Entity() | ||
| export class KospiMaster { |
There was a problem hiding this comment.
해당 엔티티가 주식 정보를 나타내는 엔티티인가요?
There was a problem hiding this comment.
네 임시로 사용하던 entity인데, 최신 erd에 맞춰서 업데이트하겠습니다.
| const extractedFile = path.join(baseDir, 'kosdaq_code.mst'); | ||
|
|
||
| await this.downloadFile( | ||
| 'https://new.real.download.dws.co.kr/common/master/kosdaq_code.mst.zip', |
There was a problem hiding this comment.
이런 url 같은것은 따로 상수나 env로 넣는 것이 좋다고 생각합니다!
| export class OpenapiScraperService { | ||
| private readonly config: typeof openApiConfig; | ||
| public constructor() { | ||
| this.config = openApiConfig; |
There was a problem hiding this comment.
7번 라인에 바로 넣지 않고 생성자에 내부에 주입을 하는 이유가 있나요?
| }); | ||
|
|
||
| for await (const row of rl) { | ||
| const shortCode = row.slice(0, 9).trim(); |
There was a problem hiding this comment.
row.slice().trim() 코드가 반복되네요. 함수로 빼면 될거 같아요
There was a problem hiding this comment.
그리고 매직넘버도 상수화 하는 것은 어떤가요?
.vscode/settings.json
Outdated
There was a problem hiding this comment.
로컬상에서 prittier 적용이 안 되서 사용하는 파일인데 .gitignore에 없나 보네요 ㅠ 추가할게요
| }); | ||
|
|
||
| for await (const row of rl) { | ||
| const shortCode = row.slice(0, 9).trim(); |
There was a problem hiding this comment.
그리고 매직넘버도 상수화 하는 것은 어떤가요?
close #61
✅ 작업 내용
📸 스크린샷(FE만)
📌 이슈 사항
🟢 완료 조건
✍ 궁금한 점
😎 체크 사항