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.

Sign up for GitHub

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

Jump to bottom

[BE] 10.03 뉴스 조회 API 구현 #181 #203

Merged
merged 17 commits into from
Nov 27, 2024
Merged

[BE] 10.03 뉴스 조회 API 구현 #181 #203

merged 17 commits into from
Nov 27, 2024

Conversation

uuuo3o
Copy link
Collaborator

@uuuo3o uuuo3o commented Nov 26, 2024

✅ 주요 작업

  • 네이버 뉴스 api를 사용해 주식, 증권 관련 뉴스를 가져옴
  • 8시 - 16시 사이 매 30분(0분, 30분)마다 api를 호출해 DB에 값을 업데이트
  • updated_at 컬럼을 통해 cron 동작 시간 확인 가능

💭 고민과 해결과정

  • DB에 시간이 밀리초까지 저장되는게 불만이었는데.. precision 옵션으로 밀리초를 없애려고 하면 에러가 떠서 아직 지우지 못했다.
  • 어떤 검색어로 주식 관련 데이터를 검색할 지 고민이어서 현재 주식, 증권만 검색해서 값을 넣어놨다.

uuuo3o added 15 commits November 25, 2024 14:08
@uuuo3o
✨ feat: 뉴스 조회 API용 파일 준비#181
bc2ec06
@uuuo3o
Merge remote-tracking branch 'refs/remotes/origin/back/main' into fea…
8c74d27 
…ture/api/news-#181

# Conflicts:
#	BE/src/app.module.ts
@uuuo3o
⚙️ chore: cheerio 설치#181
096f7f1
@uuuo3o
✨ feat: 뉴스 정보를 크롤링으로 가져오기#181
aba2d09
@uuuo3o
✨ feat: 뉴스 API에 사용할 DTO, interface 생성#181
5606380
@uuuo3o
✨ feat: 네이버 API에 사용할 domain service 구현#181
7ef29ef
@uuuo3o
🔧 fix: 잘못된 이름 수정#181
04fbabc
@uuuo3o
✨ feat: 네이버 검색 api를 활용해 검색어 별로 5개의 뉴스 정보를 받아올 수 있도록 구현#181
e33fce6
@uuuo3o
✨ feat: 뉴스 정보를 가져오는 api 구현#181
d56b508
@uuuo3o
✨ feat: 뉴스 정보 저장에 사용할repository 구현#181
4814532
@uuuo3o
✨ feat: 주기적으로 뉴스 api를 호출해 db table에 저장하고, 반환하는 코드 구현#181
b5f2cc2
@uuuo3o
Revert "⚙️ chore: cheerio 설치#181"
f193dca 
This reverts commit 096f7f1.
@uuuo3o
Merge branch 'back/main' of https://github.com/boostcampwm-2024/web16…
3d598f2 
…-JuGa into feature/api/news-#181
@uuuo3o
🚚 rename: repository 위치 변경#181
317c0ea
@uuuo3o
📝 docs: swagger 태그 추가#181
b07988d
@uuuo3o uuuo3o added BE 백엔드 API API 구현 labels Nov 26, 2024
@uuuo3o uuuo3o requested review from jinddings and sieunie November 26, 2024 13:23
@uuuo3o uuuo3o self-assigned this Nov 26, 2024
@uuuo3o uuuo3o linked an issue Nov 26, 2024 that may be closed by this pull request
[BE] 10.03 뉴스 조회 API 구현 #181
Closed
jinddings
jinddings approved these changes Nov 26, 2024
View reviewed changes
Copy link
Collaborator

@jinddings jinddings left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

BE/src/news/naver-api-domian.service.ts Outdated
};
const url = 'https://openapi.naver.com/v1/search/news.json';

console.log(params);
Copy link
Collaborator

Choose a reason for hiding this comment

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

🟢 console.log 남아 있는거 같습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

하 ㅋㅋㅋㅋㅋ 진짜 감사합니다. 콘솔이 분명 찍힐텐데 왜 눈치 못챘을까요 ..ㅎㅎ

sieunie
sieunie approved these changes Nov 27, 2024
View reviewed changes
Copy link
Collaborator

@sieunie sieunie left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!

BE/src/news/news.module.ts Outdated
Comment on lines 9 to 13
@Module({
controllers: [NewsController],
providers: [NewsService, NaverApiDomianService, NewsRepository],
imports: [TypeOrmModule.forFeature([News])],
})
Copy link
Collaborator

Choose a reason for hiding this comment

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

🟢 사소한건데 이 부분 순서도 통일하는게 예뻐 보일 것 같아용 제 기억에 아마도 다 imports, controllers, providers 순으로 되어있던 것 같습니당

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

확인했습니당 ㅎㅎㅎㅎ! 위에 콘솔 지우면서 같이 바꿀게요!!

@uuuo3o uuuo3o merged commit 4b0dda9 into back/main Nov 27, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jinddings jinddings jinddings approved these changes

@sieunie sieunie sieunie approved these changes

Assignees

@uuuo3o uuuo3o

Labels
API API 구현 BE 백엔드
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BE] 10.03 뉴스 조회 API 구현
3 participants
@uuuo3o @jinddings @sieunie