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

[BE] 이미지(만) 업로드하는 api 추가 #980

Merged
merged 9 commits into from
Jan 12, 2025

Conversation

pilyang
Copy link
Collaborator

@pilyang pilyang commented Dec 31, 2024

이슈번호

close #969

#978 merge한 이후에 확인하기!!

PR 내용

  • feed image에 status field추가
    • PENDING, APPROVED, EXPIRED
  • 추가된 api로 이미지 업로드시 PENDING상태로 이미지 업로드

주의사항

  • 앞작업 (리팩터링 [BE] 이미지 검증로직 변경 #978 먼저 머지하고 이거 머지하기)
    • 여기에 앞 pr에 있는 내용도 포함되어서 같이 보임
  • 머지하기 전에 디비스키마 변경 확인!!

Copy link

github-actions bot commented Dec 31, 2024

Test Results

160 files  160 suites   32s ⏱️
512 tests 512 ✅ 0 💤 0 ❌
516 runs  516 ✅ 0 💤 0 ❌

Results for commit c1e0201.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@SproutMJ SproutMJ left a comment

Choose a reason for hiding this comment

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

확인 완료했습니다

@pilyang
Copy link
Collaborator Author

pilyang commented Jan 12, 2025

배포전 우선 아래 스크립토러 디비마이그레이션 함
배포전 오류 발생 방지를 위해 isexpried 칼럼은 아직 안날림

START TRANSACTION;

ALTER TABLE feed_thread_image ADD COLUMN status VARCHAR(255);
UPDATE feed_thread_image SET status = 'ACTIVATED';
ALTER TABLE feed_thread_image MODIFY COLUMN feed_thread_id bigint NULL;

COMMIT;

@pilyang pilyang merged commit c87c066 into develop Jan 12, 2025
3 checks passed
@pilyang pilyang deleted the feat/be/feed-image-sep-upload branch January 12, 2025 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] 피드 이미지 업로드 API 분리
2 participants