Skip to content

Conversation

@Dimo-2562
Copy link
Contributor

❤️ 기능 설명

버그 수정: TechBlog logoUrl 조건문 오류 수정

문제:

  • TechBlog 생성자에서 logoUrl을 파라미터로 받았으나, 조건문에서 this.logoUrl을 체크하여 항상 null로 판단되는 버그
  • 이로 인해 logoUrl을 명시적으로 전달해도 무시되고 항상 Google Favicon API를 사용하는 문제 발생

해결:

  • TechBlog.java:40 - 조건문을 if (this.logoUrl != null)if (logoUrl != null)로 수정하여 파라미터 값을 올바르게 체크하도록 수정
  • InitialDataConfig.java - 네이버D2와 SK C&C의 logoUrl을 빈 문자열(" ")에서 실제 로고 URL로 변경

변경 파일:

  • src/main/java/com/techfork/domain/source/entity/TechBlog.java
  • src/main/java/com/techfork/global/config/InitialDataConfig.java

연결된 issue

연결된 issue를 자동으로 닫기 위해 아래 {이슈넘버}를 입력해주세요.

close #127



✅ 체크리스트

  • PR 제목 규칙 잘 지켰는가?
  • 추가/수정사항을 설명하였는가?
  • 테스트 결과 사진을 넣었는가?
  • 이슈넘버를 적었는가?

@Dimo-2562 Dimo-2562 self-assigned this Jan 1, 2026
@Dimo-2562 Dimo-2562 added 🎵 CHORE 잡일 📝 domain:post 게시글 관련 labels Jan 1, 2026
@Dimo-2562 Dimo-2562 merged commit 6de0bf8 into develop Jan 1, 2026
1 check passed
@Dimo-2562 Dimo-2562 deleted the chore/#127 branch January 1, 2026 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎵 CHORE 잡일 📝 domain:post 게시글 관련

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CHORE] 네이버 D2 로고 이미지 및 SK C&C url 추가

2 participants