Skip to content

[DEV] 부적절한 단어 필터링 모달 추가 및 모달 사이즈 변경#143

Merged
drddyn merged 4 commits into
developfrom
dev/blocked-words-filtering
Jun 1, 2026
Merged

[DEV] 부적절한 단어 필터링 모달 추가 및 모달 사이즈 변경#143
drddyn merged 4 commits into
developfrom
dev/blocked-words-filtering

Conversation

@drddyn

@drddyn drddyn commented May 27, 2026

Copy link
Copy Markdown
Owner

🔗 관련 이슈

close #141

✨ 요약

부적절한 단어를 필터링하는 로직을 추가했습니다.

📝 작업 내용

  • 부적절한 단어 필터링 기능 추가
  • 부적절한 단어 모달 연결
  • 모달 색상, 사이즈 변경
image image image

@drddyn drddyn self-assigned this May 27, 2026
@vercel

vercel Bot commented May 27, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
mine Ready Ready Preview, Comment Jun 1, 2026 3:41pm

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a client-side harmful keyword filtering system using a predefined blocked words list and a warning modal. It also refactors modal styles and icon color handling. The review feedback highlights a critical issue in the word filtering logic where simple substring matching causes severe false positives for common English words (e.g., "travel" blocked by "av"). Additionally, the feedback points out duplicate validation logic and leftover debugging statements in MainPage.tsx, an undefined CSS class font-16 in ConfirmModal.tsx, and a potential visibility issue with the close icon in ShareModal.tsx due to missing color classes.

Comment thread Mine/src/utils/blockedWords.ts
Comment thread Mine/src/pages/main/MainPage.tsx
Comment thread Mine/src/components/common/ConfirmModal.tsx
Comment thread Mine/src/components/hamburgerModal/ShareModal.tsx
@drddyn drddyn merged commit 8e3068b into develop Jun 1, 2026
3 checks passed
@drddyn drddyn deleted the dev/blocked-words-filtering branch June 1, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DEV] 필터링 단어 추가

1 participant