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

Merged
merged 13 commits into from
Jan 31, 2025

Conversation

PororoAndFriends
Copy link
Contributor

#️⃣연관된 이슈

ex) #33

📝작업 내용

GCS를 이용하여 동아리 관리자의 로고와 피드를 업로드 하는 기능을 구현하였음

SecurityConfig에 GCS 인증 & storage 객체를 Bean에 등록하였음
image

컨트롤러에서 로고 업데이트인지 피드 업데이트인지에 대한 정보를 받고
image
기본적인 영상 업로드 전 전처리를 해 준 후
image
uploadFile을 호출하여 중복을 최소화 하였음
image

로고 이미지 업로드 확인
스크린샷 2025-01-29 19 28 57
피드 이미지(2개) 업로드 확인
스크린샷 2025-01-29 19 28 50

@PororoAndFriends PororoAndFriends self-assigned this Jan 29, 2025
Copy link
Member

@Zepelown Zepelown left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.
리뷰 답장해주시면 감사하겠습니다!

@RestController
@RequestMapping("/api/club")
@RequiredArgsConstructor
public class GcsController {
Copy link
Member

Choose a reason for hiding this comment

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

결국 Club에 대한 도메인이라 Clubcontroller에 작성하거나, ClubImageUploadController 처럼 더 club이 기능을 강조하는 것이 어떨까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

전혀 생각 못한 부분인데 생각해 보니 그렇네요!
Clubcontroller 에서 다루는 클럽의 텍스트 데이터들을 다루는 것과 이미지 데이터를 다루는 부분은 성격이 조금 다른 것 같아서 ClubImageController 로 수정하였습니다~


@Service
@RequiredArgsConstructor
public class GcsService {
Copy link
Member

Choose a reason for hiding this comment

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

이 또한 ClubImageUploadService나 ClubImageService로 도메인을 club으로 합치면 좋지 않을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 이부분도 위와 마찬가지로 ClubImageService로 변경하였습니다

Copy link
Member

@Zepelown Zepelown left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

@PororoAndFriends PororoAndFriends added ✨ Feature 기능 개발 💾 BE Backend labels Jan 31, 2025
Copy link
Member

@Zepelown Zepelown left a comment

Choose a reason for hiding this comment

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

Conflict 해결 확인했습니다~

@PororoAndFriends PororoAndFriends merged commit c503c59 into develop/be Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💾 BE Backend ✨ Feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants