search

FC-DEV-FinalProject / FinalProject_2VEN_FE

데브코스 파이널프로젝트 시스메틱 2조 이븐한조 FE 리포지토리
0 stars 1 forks source link

fix: 닉네임 중복 검사 로직 수정 (#182) #183

Closed clara-shin closed 10 hours ago

clara-shin commented 10 hours ago

🚀 풀 리퀘스트 제안

📋 작업 내용

정규식 패턴 재정의:

유효성 검사 로직:

가능한 모든 조합 허용:

✅ 다음과 같은 닉네임을 모두 허용

🔧 변경 사항

주요 변경 사항을 요약해 주세요.

📸 스크린샷 (선택 사항)

수정된 화면 또는 기능을 시연할 수 있는 스크린샷을 첨부해 주세요.

📄 기타

추가적으로 전달하고 싶은 내용이나 특별한 요구 사항이 있으면 작성해 주세요.

Sourcery에 의한 요약

버그 수정:

Original summary in English ## Summary by Sourcery Bug Fixes: - Fix nickname validation logic to allow a wider range of character combinations, including Korean characters, uppercase and lowercase English letters, and numbers.
netlify[bot] commented 10 hours ago

Deploy Preview for sysmetics ready!

Name Link
Latest commit 570c589530b74f60b2e096ec3bfc2f17b127d0d6
Latest deploy log https://app.netlify.com/sites/sysmetics/deploys/67404948f7926a00089a4a46
Deploy Preview https://deploy-preview-183--sysmetics.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

sourcery-ai[bot] commented 10 hours ago

리뷰어 가이드 by Sourcery

이 PR은 별명 유효성 검사 로직을 수정하여 한국어 문자, 대문자 및 소문자 영어 문자, 숫자를 포함한 보다 유연한 문자 세트를 지원합니다. 구현은 이러한 문자 유형의 조합을 허용하면서 길이 및 공백 제한을 유지하기 위해 정규식 패턴과 유효성 검사 로직을 모두 변경합니다.

별명 유효성 검사를 위한 업데이트된 클래스 다이어그램

classDiagram
    class REGEX {
        +korean: RegExp
        +upperEnglish: RegExp
        +lowerEnglish: RegExp
        +number: RegExp
        +allowedChars: RegExp
    }
    class VALIDATION_MESSAGE {
        +nickname: Object
    }
    class Validation {
        +validateNickname(nickname: string): Object
    }
    REGEX --> Validation
    VALIDATION_MESSAGE --> Validation
    note for REGEX "별명 유효성 검사를 위한 업데이트된 정규식 패턴"
    note for Validation "새로운 문자 세트를 지원하기 위한 업데이트된 유효성 검사 로직"

파일 수준 변경 사항

변경 사항 세부 사항 파일
별명 유효성 검사를 위한 정규식 패턴 재정의
  • 한국어 문자 유효성 검사를 위한 패턴 추가
  • 영어 문자 유효성 검사를 대문자와 소문자 패턴으로 분리
  • 숫자 유효성 검사 패턴 업데이트
  • 전체 문자 세트를 유효성 검사하기 위한 새로운 allowedChars 패턴 추가
 src/constants/auth.ts
별명 유효성 검사 로직 업데이트
  • 적어도 하나의 유효한 문자 유형을 확인하도록 문자 유형 유효성 검사 수정
  • 문자와 숫자 모두 필요하다는 요구 사항 제거
  • 새로운 문자 세트 규칙을 반영하도록 오류 메시지 업데이트
  • 기존 길이(2-10자) 및 공백 제한 유지
 src/utils/validation.ts

관련된 문제일 가능성

팁 및 명령어 #### Sourcery와 상호작용하기 - **새로운 리뷰 트리거:** 풀 리퀘스트에 `@sourcery-ai review`라고 댓글을 남깁니다. - **토론 계속하기:** Sourcery의 리뷰 댓글에 직접 답장합니다. - **리뷰 댓글에서 GitHub 이슈 생성:** 리뷰 댓글에서 이슈를 생성하도록 Sourcery에 요청합니다. - **풀 리퀘스트 제목 생성:** 풀 리퀘스트 제목 어디에나 `@sourcery-ai`를 작성하여 언제든지 제목을 생성합니다. - **풀 리퀘스트 요약 생성:** 풀 리퀘스트 본문 어디에나 `@sourcery-ai summary`를 작성하여 언제든지 PR 요약을 생성합니다. 이 명령을 사용하여 요약이 삽입될 위치를 지정할 수도 있습니다. #### 경험 맞춤화하기 [대시보드](https://app.sourcery.ai)에 액세스하여: - Sourcery가 생성한 풀 리퀘스트 요약, 리뷰어 가이드 등과 같은 리뷰 기능을 활성화하거나 비활성화합니다. - 리뷰 언어를 변경합니다. - 사용자 정의 리뷰 지침을 추가, 제거 또는 편집합니다. - 기타 리뷰 설정을 조정합니다. #### 도움 받기 - 질문이나 피드백이 있는 경우 [지원 팀에 문의](mailto:support@sourcery.ai)하세요. - 자세한 가이드와 정보를 보려면 [문서](https://docs.sourcery.ai)를 방문하세요. - [X/Twitter](https://x.com/SourceryAI), [LinkedIn](https://www.linkedin.com/company/sourcery-ai/) 또는 [GitHub](https://github.com/sourcery-ai)에서 Sourcery 팀을 팔로우하여 소식을 받아보세요.
Original review guide in English ## Reviewer's Guide by Sourcery This PR modifies the nickname validation logic to support a more flexible character set including Korean characters, uppercase and lowercase English letters, and numbers. The implementation changes both the regex patterns and validation logic to allow any combination of these character types while maintaining length and whitespace restrictions. #### Updated class diagram for nickname validation ```mermaid classDiagram class REGEX { +korean: RegExp +upperEnglish: RegExp +lowerEnglish: RegExp +number: RegExp +allowedChars: RegExp } class VALIDATION_MESSAGE { +nickname: Object } class Validation { +validateNickname(nickname: string): Object } REGEX --> Validation VALIDATION_MESSAGE --> Validation note for REGEX "Updated regex patterns for nickname validation" note for Validation "Updated validation logic to support new character set" ``` ### File-Level Changes | Change | Details | Files | | ------ | ------- | ----- | | Redefined regex patterns for nickname validation |
  • Added pattern for Korean character validation
  • Split English character validation into uppercase and lowercase patterns
  • Updated number validation pattern
  • Added new allowedChars pattern to validate overall character set
| `src/constants/auth.ts` | | Updated nickname validation logic |
  • Modified character type validation to check for at least one valid character type
  • Removed requirement for both letters and numbers
  • Updated error messages to reflect new character set rules
  • Maintained existing length (2-10 characters) and whitespace restrictions
| `src/utils/validation.ts` | ### Possibly linked issues - **#182**: PR refines nickname validation logic, directly addressing the issue's requirements. - **#182**: PR refines nickname validation logic, directly related to issue's nickname validation task. ---
Tips and commands #### Interacting with Sourcery - **Trigger a new review:** Comment `@sourcery-ai review` on the pull request. - **Continue discussions:** Reply directly to Sourcery's review comments. - **Generate a GitHub issue from a review comment:** Ask Sourcery to create an issue from a review comment by replying to it. - **Generate a pull request title:** Write `@sourcery-ai` anywhere in the pull request title to generate a title at any time. - **Generate a pull request summary:** Write `@sourcery-ai summary` anywhere in the pull request body to generate a PR summary at any time. You can also use this command to specify where the summary should be inserted. #### Customizing Your Experience Access your [dashboard](https://app.sourcery.ai) to: - Enable or disable review features such as the Sourcery-generated pull request summary, the reviewer's guide, and others. - Change the review language. - Add, remove or edit custom review instructions. - Adjust other review settings. #### Getting Help - [Contact our support team](mailto:support@sourcery.ai) for questions or feedback. - Visit our [documentation](https://docs.sourcery.ai) for detailed guides and information. - Keep in touch with the Sourcery team by following us on [X/Twitter](https://x.com/SourceryAI), [LinkedIn](https://www.linkedin.com/company/sourcery-ai/) or [GitHub](https://github.com/sourcery-ai).