🔁 포트폴리오 URL이 비어있는 상황에서는 무조건 빈 문자열을 반환함 #407

enbraining · 2024-07-15T04:14:54Z

💡 배경 및 개요

포트폴리오 URL이 비어있는 경우 포트폴리오 파일 URL을 포트폴리오 URL로 반환하던 것을 빈 문자열을 반환하는 것으로 수정했습니다.

Resolves: #406

✅ PR 체크리스트

템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요!

이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. .env, 노션, README)
이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "환경값 추가되었어요")
작업한 코드가 정상적으로 동작하나요?
Merge 대상 브랜치가 올바른가요?
PR과 관련 없는 작업이 있지는 않나요?

ani2689 · 2024-07-15T04:43:32Z

...ore/src/main/kotlin/team/msg/sms/domain/user/usecase/QueryCurrentUserProfileDetailUseCase.kt

            !student.portfolioUrl.isNullOrBlank() -> student.portfolioUrl
-            !student.portfolioFileUrl.isNullOrBlank() -> student.portfolioFileUrl
            else -> ""


조건식이 두개 뿐이라면 when을 사용할 필요 없이 if를 사용해주는게 좋을 것 같아요

변경했습니다!

♻️ 포트폴리오 URL이 비어있는 상황에서는 무조건 빈 문자열을 반환함

36c2938

enbraining added 1️⃣ Priority: High 우선순위 - 상 ♻️ Refactor 코드 리팩토링 labels Jul 15, 2024

enbraining requested review from louis7308, Huuuunee and ani2689 July 15, 2024 04:14

enbraining self-assigned this Jul 15, 2024

enbraining linked an issue Jul 15, 2024 that may be closed by this pull request

포트폴리오 URL에 포트폴리오 파일 URL을 보내지 않도록 변경 #406

Closed

enbraining added 2 commits July 15, 2024 13:42

♻️ 포트폴리오 URL이 비어있는 상황에서는 무조건 빈 문자열을 반환함

6a6ba91

🔥 잘못 올린 파일 제거

4ecc402

ani2689 reviewed Jul 15, 2024

View reviewed changes

♻️ if문으로 변경

fe0d566

ani2689 approved these changes Jul 15, 2024

View reviewed changes

enbraining merged commit 9ca0e77 into develop Jul 15, 2024
1 check passed

enbraining deleted the hotfix/406-exclude-portfolio-file-url branch July 15, 2024 04:47

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🔁 포트폴리오 URL이 비어있는 상황에서는 무조건 빈 문자열을 반환함 #407

🔁 포트폴리오 URL이 비어있는 상황에서는 무조건 빈 문자열을 반환함 #407

enbraining commented Jul 15, 2024 •

edited

Loading

ani2689 Jul 15, 2024

enbraining Jul 15, 2024

🔁 포트폴리오 URL이 비어있는 상황에서는 무조건 빈 문자열을 반환함 #407

🔁 포트폴리오 URL이 비어있는 상황에서는 무조건 빈 문자열을 반환함 #407

Conversation

enbraining commented Jul 15, 2024 • edited Loading

💡 배경 및 개요

✅ PR 체크리스트

ani2689 Jul 15, 2024

Choose a reason for hiding this comment

enbraining Jul 15, 2024

Choose a reason for hiding this comment

enbraining commented Jul 15, 2024 •

edited

Loading