Skip to content

Conversation

@yeop-sang
Copy link
Contributor

@yeop-sang yeop-sang commented Aug 18, 2025

#️⃣ Issue Number

📝 요약(Summary)

veo3와 s3 업로드 기능 완성

💬 공유사항 to 리뷰어

app/schemas/veo.py 및 app/tasks/veo_tasks.py 가 주요 veo 관련 코드

test/* 및 script/* 는 테스트 환경을 위한 간단한 코드 (local 혹은 실행방법 확인용)
main commit 시 drop 할 예정

Celery를 실행시켜야 비동기가 정상작동함 !!반드시 확인바람!!

celery -A app.celery_app worker --loglevel=info --concurrency=4
필요시 dockerfile 제작 가능

기존 코드 수정 (!! 반드시 확인 바람)

  1. app/main.py :load_dotenv(dotenv_path=Path(__file__).resolve().parent / ".env") 가 로컬에서 오류를 일으켜 load_dotenv()로 변경시 문제가 있는지 확인 바람
  2. app/schemas/sentences.py 에서 계속 linter 오류가 발생 확인 바람

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • [] 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

과금이 너무 불안해서 아직 전체 워크로드 테스트 하지 않음.
reject 시 요청 바람

@yeop-sang yeop-sang requested a review from minhyuk2 August 18, 2025 15:33
@yeop-sang yeop-sang changed the title Feat/sangyeop feat: veo3와 s3 업로드 기능 완성 Aug 18, 2025
@yeop-sang yeop-sang self-assigned this Aug 18, 2025
@yeop-sang yeop-sang added the enhancement New feature or request label Aug 18, 2025
@minhyuk2 minhyuk2 merged commit e75e908 into dev Aug 19, 2025
@minhyuk2 minhyuk2 deleted the feat/sangyeop branch August 19, 2025 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants