Skip to content

Fix/#135: 폴더명 글자수 제한 10자로 수정#137

Merged
Darren4641 merged 3 commits intostagingfrom
fix/#135
Feb 14, 2026
Merged

Fix/#135: 폴더명 글자수 제한 10자로 수정#137
Darren4641 merged 3 commits intostagingfrom
fix/#135

Conversation

@koosco
Copy link
Member

@koosco koosco commented Feb 13, 2026

summary

  • 디자인 변경에 따라 폴더명 글자수 제한을 16 -> 10 변경

@koosco koosco requested a review from Darren4641 February 13, 2026 16:38
@koosco koosco marked this pull request as ready for review February 13, 2026 16:38
@gemini-code-assist
Copy link

Summary of Changes

Hello @koosco, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 Pull Request는 디자인 변경 요구사항에 따라 폴더명의 최대 허용 글자수를 조정하고, 이에 대한 유효성 검사 로직과 관련 테스트를 업데이트합니다. 사용자 경험을 개선하고 시스템의 일관성을 유지하기 위해 폴더명 입력에 대한 새로운 제한을 적용합니다.

Highlights

  • 폴더명 글자수 제한 변경: 폴더 생성 및 업데이트 요청 시 폴더명의 최대 글자수가 16자에서 10자로 변경되었습니다.
  • 유효성 검사 메시지 업데이트: 변경 된 글자수 제한에 맞춰 유효성 검사 메시지가 "폴더명은 10글자 이하여야 합니다."로 수정되었습니다.
  • E2E 테스트 추가: 10자를 초과하는 폴더명에 대한 400 에러 응답을 확인하는 E2E 테스트가 CreateFolderE2ETestUpdateFolderE2ETest에 각각 추가되었습니다.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Changelog
  • src/main/kotlin/com/yapp2app/photo/api/dto/FolderRequest.kt
    • CreateFolderRequestUpdateFolderRequest DTO에서 name 필드의 @Size 어노테이션 max 값을 16에서 10으로 변경했습니다.
    • @Size 어노테이션의 message 속성을 "폴더명은 1 ~ 16자 사이여야 합니다."에서 "폴더명은 10글자 이하여야 합니다."로 업데이트했습니다.
  • src/test/kotlin/com/yapp2app/e2e/photo/folder/CreateFolderE2ETest.kt
    • 폴더명이 10자를 초과할 때 400 Bad Request 에러를 반환하는지 확인하는 givenTooLongFolderName_whenCreateFolder_thenReturnsBadRequest 테스트 케이스를 추가했습니다.
  • src/test/kotlin/com/yapp2app/e2e/photo/folder/UpdateFolderE2ETest.kt
    • 폴더명이 10자를 초과할 때 400 Bad Request 에러를 반환하는지 확인하는 givenTooLongFolderName_whenUpdateFolder_thenReturnsBadRequest 테스트 케이스를 추가했습니다.
Activity
  • 이 Pull Request에 대한 코멘트, 리뷰 또는 진행 상황과 같은 추가적인 활동은 아직 없습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@github-actions
Copy link

Code Format Check ✅ PASSED

Spotless Check: success

✨ All code formatting checks passed!


Pushed by: @koosco, Action: pull_request

Copy link

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

Choose a reason for hiding this comment

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

Code Review

이 PR은 폴더명의 글자 수 제한을 16자에서 10자로 변경하는 수정사항을 담고 있습니다. DTO의 유효성 검사 규칙을 수정하고, 이에 대한 E2E 테스트 케이스를 추가한 점이 좋습니다. 전반적으로 변경 사항은 명확하며, 의도한 대로 잘 동작할 것으로 보입니다. 한 가지 제안 사항으로, 유효성 검사 메시지를 조금 더 명확하게 개선하는 것을 고려해볼 수 있습니다. 자세한 내용은 아래 주석을 참고해주세요.

@github-actions
Copy link

Test ✅ PASSED

Test Result: success

✨ All tests passed!


Pushed by: @koosco, Action: pull_request

@koosco koosco changed the title fix: 폴더명 글자수 제한 10자로 수정 Fix/#135: 폴더명 글자수 제한 10자로 수정 Feb 13, 2026
@github-actions
Copy link

Code Format Check ✅ PASSED

Spotless Check: success

✨ All code formatting checks passed!


Pushed by: @koosco, Action: pull_request

@github-actions
Copy link

Test ✅ PASSED

Test Result: success

✨ All tests passed!


Pushed by: @koosco, Action: pull_request

Copy link
Member

@Darren4641 Darren4641 left a comment

Choose a reason for hiding this comment

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

flyway에서도 alter 명령어로 length 사이즈 줄이몀ㄴ 좋을 것 같습니다! 개발서버에 이미 글자수 10자 넘은거는 10자 아래로 수정한 뒤 진행하면 좋을 것 같아요!

@github-actions
Copy link

Code Format Check ✅ PASSED

Spotless Check: success

✨ All code formatting checks passed!


Pushed by: @koosco, Action: pull_request

@github-actions
Copy link

Test ✅ PASSED

Test Result: success

✨ All tests passed!


Pushed by: @koosco, Action: pull_request

Copy link
Member

@Darren4641 Darren4641 left a comment

Choose a reason for hiding this comment

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

LGTM!

@Darren4641 Darren4641 merged commit 4e0d107 into staging Feb 14, 2026
2 checks passed
@Darren4641 Darren4641 deleted the fix/#135 branch February 14, 2026 10:01
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.

2 participants