Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🔀 :: (#1258) 닉네임 입력 조건 추가 #1274

Merged
merged 4 commits into from
Sep 1, 2024

Conversation

KangTaeHoon
Copy link
Contributor

@KangTaeHoon KangTaeHoon commented Sep 1, 2024

💡 배경 및 개요

  • 닉네임, 플리 이름 입력시 마지막 글자에서 한글 자모 조합이 되지 않는 문제 발생

Resolves: #1258

📃 작업내용

  • 현재 작성한 조건(백스페이스, 글자수 제한)에서 입력한 '마지막' 글자가 완성된 한글이 아니거나, 추가로 자음이 결합될 수 있는 조건을 추가

🙋‍♂️ 리뷰노트

  • 이런 조건을 추가했을때, 마지막 글자가 ex: '일' 이라면 추가로 받침이 더 올 수 있으므로 글자 수 제한을 넘어 계속 입력이 가능합니다. 하지만 빨간 경고 UI를 띄워 보여주고 있으므로 괜찮을듯 함.

✅ PR 체크리스트

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

🎸 기타

Copy link

github-actions bot commented Sep 1, 2024

✅ Successful finished SwiftLint

Copy link

github-actions bot commented Sep 1, 2024

✅ Assign 자동 지정을 성공했어요!

@KangTaeHoon

@KangTaeHoon KangTaeHoon merged commit e7638e4 into release/3.0.0 Sep 1, 2024
3 checks passed
@KangTaeHoon KangTaeHoon deleted the 1258-bug-fix-naming-input-condition branch September 1, 2024 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0️⃣ Priority: Critical 우선순위 긴급 🐞 Bug 버그 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants