Skip to content

Comments

fix: 단어 생성시 메모 빈칸 이슈 수정#4

Merged
goodaseul merged 1 commit intodevfrom
main
Jan 19, 2026
Merged

fix: 단어 생성시 메모 빈칸 이슈 수정#4
goodaseul merged 1 commit intodevfrom
main

Conversation

@goodaseul
Copy link
Owner

@goodaseul goodaseul commented Jan 19, 2026

단어 생성시 메모에 빈칸이 없을 때도 filter 메모있음 으로 분류가 되는 이슈를 수정

Summary by CodeRabbit

  • Bug Fixes
    • Fixed handling of empty memo fields when uploading words to ensure consistent data storage.

✏️ Tip: You can customize this high-level summary in your review settings.

@goodaseul goodaseul self-assigned this Jan 19, 2026
@goodaseul goodaseul added the bug Something isn't working label Jan 19, 2026
@vercel
Copy link

vercel bot commented Jan 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
stack-plus Ready Ready Preview, Comment Jan 19, 2026 11:11am

@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The uploadWords function in src/api/words.ts now normalizes the memo field before insertion. Empty strings are converted to null after trimming, ensuring consistent null representation instead of storing empty values.

Changes

Cohort / File(s) Summary
Memo normalization logic
src/api/words.ts
Added normalization for memo field in uploadWords: empty strings after trim are converted to null; otherwise memo is preserved as-is during word insertion

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A memo once empty now sleeps as null,
Trimmed strings find their rest,
No more blank whispers in the database,
Just tidy, normalized best!


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@goodaseul goodaseul merged commit bdcbd7b into dev Jan 19, 2026
3 of 4 checks passed
@goodaseul goodaseul deleted the main branch January 19, 2026 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant