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

Feature/#34 script generator 구현 #35

Merged
merged 4 commits into from
Feb 20, 2024

Conversation

hong-sile
Copy link
Collaborator

@hong-sile hong-sile commented Feb 20, 2024

#️⃣ 연관된 이슈
close : #34

📝 작업 내용
기존에 존재하는 베트남 script generator를 삭제하고,
trasnlationClient를 이용한 script generator 구현

예상 소요 시간 및 실제 소요 시간
예상 소요 시간 : 3시간
실제 소요 시간 : 1시간

상세 내용

  • 코드 스타일 수정
  • scriptGeneratorComposite 삭제
  • scriptGenrator 추가

리팩터링 하고 싶은 사항

  • FakeOcrReader가 대체될 경우 테스트 패키지로 이동 재구성된 메뉴판 반환 api 구현 #12 이후 작업하기
  • FoodScraper를 MockBean으로 구현하였는데 ocrReader와 fixture를 어떻게 통일시킬지 고민하기

@hong-sile hong-sile merged commit 434dbd4 into main Feb 20, 2024
1 check passed
@hong-sile hong-sile deleted the Feature/#34-script_generator_구현 branch February 21, 2024 08:32
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.

script generator 구현
1 participant