Skip to content
This repository has been archived by the owner on Mar 26, 2024. It is now read-only.

[Bugfix] originalText null 방지 코드 추가 #220

Merged
merged 1 commit into from
Jan 27, 2024

Conversation

happymink
Copy link
Contributor

@happymink happymink commented Jan 27, 2024

# Google Text가 Null일 경우, 공백으로 대체합니다.

리뷰는 존재하지만 해당 리뷰의 본문이 공백인 경우, NPE가 발생하였습니다.
해당 사항에 대한 Bugfix를 진행합니다.

변경사항

Review를 변환하는 도중에서 본문이 없는 리뷰에 대한 NPE 방지 코드를 작성합니다.

Issue Link - #218

특이사항

파악한 바로는 구글 리뷰 작성 정책상, 공백으로 기재할 수 있는 항목은 본문 하나인 것 같아 해당 케이스만 고려했습니다.

참고

리뷰작성

`#` Google Text가 Null일 경우, 공백으로 지정합니다.
@happymink happymink added the bug Something isn't working label Jan 27, 2024
@happymink happymink requested review from laigasus and Dr-KoKo January 27, 2024 21:20
@happymink happymink self-assigned this Jan 27, 2024
Copy link

Test Results

37 tests  ±0   37 ✅ ±0   2s ⏱️ ±0s
15 suites ±0    0 💤 ±0 
15 files   ±0    0 ❌ ±0 

Results for commit 0010f22. ± Comparison against base commit d27bf56.

Copy link
Member

@laigasus laigasus left a comment

Choose a reason for hiding this comment

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

검증되었습니다!
이상없습니다!

@happymink happymink merged commit d42d1ba into develop Jan 27, 2024
3 checks passed
@happymink happymink deleted the bugfix/#218-review-original-text-bugfix branch January 27, 2024 22:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants