Skip to content

fix: 웹 동아리 한 줄 소개를 100자로 확장#668

Merged
JanooGwan merged 3 commits into
developfrom
fix/web-club-description-length
Jun 22, 2026
Merged

fix: 웹 동아리 한 줄 소개를 100자로 확장#668
JanooGwan merged 3 commits into
developfrom
fix/web-club-description-length

Conversation

@JanooGwan

@JanooGwan JanooGwan commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

📋 개요

  • 웹 동아리 한 줄 소개가 30자에서 강제로 잘리는 문제를 해결하기 위해 길이 제한을 100자로 확장합니다.

🔧 주요 변경 내용

  • 기존의 동아리 한 줄 소개 30자 제한으로 인해 크롤링했던 내용이 잘리는 현상이 있어 우선 100자로 늘렸습니다.

📝 참고 사항


✅ Checklist (완료 조건)

  • 코드 스타일 / 가이드 준수
  • 테스트 코드 포함
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증(API 키, 환경 변수, 개인정보 등)

@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

동아리 한 줄 소개(description/shortDescription) 필드의 최대 허용 길이를 30자에서 100자로 확장합니다. DB 마이그레이션, JPA 엔티티 @Column 제약, DTO @Size 검증, OpenAPI @Schema 문서, 서비스 내부 상수가 일괄 변경되며, 새 경계값을 검증하는 통합·단위 테스트가 추가됩니다.

Changes

한 줄 소개 최대 길이 30 → 100 확장

Layer / File(s) Summary
DB 마이그레이션 및 엔티티 컬럼 길이 확장
src/main/resources/db/migration/V89__expand_web_club_short_description.sql, src/main/java/gg/agit/konect/domain/website/model/WebClub.java, src/main/java/gg/agit/konect/domain/club/model/ClubRegistrationRequest.java, src/main/java/gg/agit/konect/domain/club/model/ClubInformationUpdateRequest.java
Flyway V89 마이그레이션으로 web_club.description, club_registration_request.short_description, club_information_update_request.short_description 컬럼을 VARCHAR(100) NOT NULL로 변경하고, 각 JPA 엔티티의 @Column(length=...) 값을 100으로 맞춥니다.
DTO 검증·OpenAPI 문서 및 서비스 상수 갱신
src/main/java/gg/agit/konect/domain/club/dto/ClubRegistrationRequestDto.java, src/main/java/gg/agit/konect/domain/club/dto/ClubInformationUpdateRequestDto.java, src/main/java/gg/agit/konect/admin/website/dto/AdminWebsiteClubSheetImportConfirmRequest.java, src/main/java/gg/agit/konect/admin/website/service/AdminWebsiteClubSheetImportService.java
shortDescription/description에 적용된 @Size(max=30)@Size(max=100)으로, @Schema 설명 문구를 "최대 100자"로 갱신하고, DESCRIPTION_MAX_LENGTH 상수를 100으로 변경합니다.
통합 테스트 및 단위 테스트 추가
src/test/java/gg/agit/konect/integration/domain/club/ClubRegistrationRequestApiTest.java, src/test/java/gg/agit/konect/unit/admin/website/service/AdminWebsiteClubSheetImportServiceTest.java
동아리 등록·정보 수정 API에 100자 경계값 테스트(성공 201, 101자 초과 시 400)를 추가하고, previewClubs에서 100자 description이 잘리지 않고 그대로 반환되는지 단위 테스트로 검증합니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

  • BCSDLab/KONECT_BACK_END#636: 동아리 등록 요청 API/DTO/엔티티(shortDescription 검증·스키마)를 최초 도입한 PR으로, 이번 PR에서 같은 지점의 @Size 제약을 직접 수정합니다.
  • BCSDLab/KONECT_BACK_END#642: ClubInformationUpdateRequestDto.shortDescription@Size(max=30) 검증·스키마를 최초 도입한 PR로, 이번 PR이 해당 값을 100으로 확장합니다.
  • BCSDLab/KONECT_BACK_END#661: AdminWebsiteClubSheetImportService의 description 길이 제한 로직과 WebClub.description 처리 흐름을 도입한 PR로, 이번 PR이 동일 코드 경로의 상한을 확장합니다.

Poem

🐰 토끼가 소개글을 쓰려 했더니,
30자론 턱없이 부족했다네.
팡팡 늘려 100자 넉넉하게,
동아리 자랑 마음껏 써봐요!
✨ 글자 수 걱정 이젠 없어요~ 🎉

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 변경 내용의 핵심을 명확하게 설명하고 있습니다. 웹 동아리 한 줄 소개 길이를 30자에서 100자로 확장하는 주요 변경사항을 정확하게 반영하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed PR 설명이 변경사항과 관련성이 있으며 명확한 목표를 제시하고 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/web-club-description-length

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@JanooGwan JanooGwan added the DB DB 마이그레이션을 위한 라벨입니다. label Jun 22, 2026
@JanooGwan JanooGwan self-assigned this Jun 22, 2026
@github-actions

Copy link
Copy Markdown

🧪 JaCoCo Coverage Report (Changed Files)

Summary

  • Overall Coverage: 84.8% ✅
  • Covered Lines: 179 / 211
  • Changed Files: 7

Coverage by File

Class Coverage Lines Status
AdminWebsiteClubSheetImportService
gg.agit.konect.admin.website.service
84.8% 179/211

📊 View Workflow Run

@JanooGwan JanooGwan merged commit 73506d0 into develop Jun 22, 2026
5 checks passed
@JanooGwan JanooGwan deleted the fix/web-club-description-length branch June 22, 2026 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DB DB 마이그레이션을 위한 라벨입니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant