Skip to content

refactor: QA 텍스트 수정#330

Merged
cho-in-sik merged 7 commits intomainfrom
refactor/326-QA-text-modification
Feb 27, 2026
Merged

refactor: QA 텍스트 수정#330
cho-in-sik merged 7 commits intomainfrom
refactor/326-QA-text-modification

Conversation

@jskimcs
Copy link
Collaborator

@jskimcs jskimcs commented Feb 27, 2026

관련 이슈

작업 분류

  • 버그 수정
  • 신규 기능 추가
  • 프로젝트 구조 변경
  • 코드 스타일 변경
  • 기존 기능 개선
  • 문서 수정

PR을 통해 해결하려는 문제가 무엇인가요? 🚀

  • 기획자, 디자이너 질문 글자 수 수정
  • 기타 질문 사항 문구 수정
  • 면접 날짜 수정
  • 개인정보수집 기간 수정
  • 이메일 작성 페이지 문구 띄어쓰기 수정
  • 포트폴리오 페이지 문구 수정
  • 지원 완료 페이지 날짜 수정

PR에서 핵심적으로 변경된 부분이 어떤 부분인가요? 👀

핵심 변경사항 이외 추가적으로 변경된 사항이 있나요? ➕

추가적으로, 리뷰어가 리뷰하며 알아야 할 정보가 있나요? 🙌

이런 부분을 신경써서 봐주셨으면 좋겠어요. 🙋🏻‍♂️

체크리스트 ✅

  • reviewers 설정
  • assignees 설정
  • label 설정

@jskimcs jskimcs self-assigned this Feb 27, 2026
@jskimcs jskimcs added the refactor 리팩터링 관련 이슈 label Feb 27, 2026
@gemini-code-assist
Copy link

Summary of Changes

Hello, 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!

이 PR은 에코노베이션 31기 신입회원 모집을 위한 QA 텍스트를 수정하고 애플리케이션 관련 날짜 및 상수를 업데이트합니다. 지원서 양식의 여러 문구와 안내사항을 명확히 하고, 텍스트 입력 필드의 최대 길이를 조정하여 사용자 경험을 개선하는 데 중점을 두었습니다.

Highlights

  • 모집 일정 및 면접 날짜 업데이트: SMS 메시지, 면접 시간표, 모집 시작/마감일, 1차 합격자 공지일, 최종 모집 마감일 등 애플리케이션 전반에 걸쳐 2026년 1학기 기준으로 날짜 정보가 업데이트되었습니다.
  • 지원서 양식 텍스트 및 문구 수정: 포트폴리오 제출 안내, 이메일 입력, 학업 외 활동 질문, 기획자 프로젝트 기획서 제출 안내 등 여러 지원서 질문의 제목과 부제목 문구가 명확하고 상세하게 수정되었습니다.
  • 텍스트 영역 최대 길이 상수 및 로직 개선: 텍스트 영역의 최대 길이를 정의하는 상수가 MAX_TEXT_LENGTH에서 MAX_BOOLEAN_TEXT_LENGTH로 변경되었고, 특정 텍스트 영역에 대한 최대 길이 설정 로직이 개선되었습니다.
  • 애플리케이션 상수 Import 경로 변경: 디자이너, 개발자, 기획자 역할에 대한 애플리케이션 상수의 import 경로가 30기에서 31기 관련 파일로 업데이트되었습니다.

🧠 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
  • frontend/app/application/backup/route.ts
    • SMS 메시지의 모집 기간 및 면접 날짜를 2026년 1학기 기준으로 업데이트했습니다.
  • frontend/app/application/page.tsx
    • 애플리케이션 상수의 import 경로를 30기에서 31기 관련 파일로 변경했습니다.
  • frontend/components/application/applicationLayout/BooleanTextarea.tsx
    • 텍스트 영역의 최대 길이를 정의하는 상수를 MAX_TEXT_LENGTH에서 MAX_BOOLEAN_TEXT_LENGTH로 변경하고 적용했습니다.
  • frontend/components/application/applicationNode/Text.component.tsx
    • 부제목 텍스트 컴포넌트에 whitespace-pre-line 클래스를 추가하여 줄 바꿈을 유지하도록 했습니다.
  • frontend/components/application/applicationNode/Textarea.component.tsx
    • MAX_TEXT_LENGTH 상수를 제거했습니다.
    • 특정 텍스트 영역의 최대 길이를 동적으로 설정하는 로직을 수정했습니다.
  • frontend/src/constants/application/31.ts
    • 전공 입력 질문의 서식을 정리했습니다.
    • 학업 외 활동 질문의 부제목 문구를 수정했습니다.
    • 면접 시간표, 모집 시작/마감일, 1차 합격자 공지일, 최종 모집 마감일을 2026년 1학기 기준으로 업데이트했습니다.
  • frontend/src/constants/application/31/designer.ts
    • 디자이너 포트폴리오 질문의 제목과 부제목 문구를 상세화했습니다.
    • 이메일 입력 질문의 띄어쓰기를 수정했습니다.
  • frontend/src/constants/application/31/developer.ts
    • 개발자 포트폴리오 질문의 제목과 부제목 문구를 상세화했습니다.
    • 이메일 입력 질문의 띄어쓰기를 수정했습니다.
  • frontend/src/constants/application/31/manager.ts
    • 기획자 포트폴리오 질문의 제목과 부제목 문구를 상세화했습니다.
    • 기획자 프로젝트 기획서 제출 안내 문구를 수정하여 참고용임을 명확히 했습니다.
    • 이메일 입력 질문의 띄어쓰기를 수정했습니다.
  • frontend/src/constants/index.ts
    • MAX_TEXT_LENGTH 상수의 이름을 MAX_BOOLEAN_TEXT_LENGTH로 변경하고 값을 1000에서 800으로 수정했습니다.
Activity
  • 현재까지 풀 리퀘스트에 대한 추가적인 활동(댓글, 리뷰 등)은 없습니다.
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.

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

전반적으로 QA에서 발견된 텍스트와 날짜들을 수정하는 변경 사항들이 잘 반영되었습니다. 몇 가지 코드 가독성 및 유지보수성 향상을 위한 제안 사항을 남깁니다. 특히, 날짜 관련 상수를 다루는 방식의 일관성을 높이고, 여러 곳에서 사용되는 매직 넘버와 매직 스트링을 상수로 추출하여 관리하는 것을 고려해보시면 좋겠습니다.

Comment on lines 201 to 213
time: [
{
startTime: new Date(2025, 8, 15, 10, 0, 0),
endTime: new Date(2025, 8, 15, 20, 30, 0),
startTime: new Date(2026, 2, 17, 10, 0, 0),
endTime: new Date(2026, 2, 17, 20, 30, 0),
},
{
startTime: new Date(2025, 8, 16, 10, 0, 0),
endTime: new Date(2025, 8, 16, 20, 30, 0),
startTime: new Date(2026, 2, 18, 10, 0, 0),
endTime: new Date(2026, 2, 18, 20, 30, 0),
},
{
startTime: new Date(2025, 8, 17, 10, 0, 0),
endTime: new Date(2025, 8, 17, 20, 30, 0),
startTime: new Date(2026, 2, 19, 10, 0, 0),
endTime: new Date(2026, 2, 19, 20, 30, 0),
},

Choose a reason for hiding this comment

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

high

APPLICATION_TIMELINE에서 new Date() 생성자에 월(month)을 0부터 시작하는 숫자 리터럴(e.g., 2 for March)로 직접 전달하고 있습니다. 하지만 파일 내 다른 날짜 상수들(START_DATE, END_DATE 등)은 1-12 범위의 월을 사용하고, Date 객체 생성 시 -1을 해주는 방식을 사용하고 있습니다. 이러한 불일치는 혼란을 야기하고 버그를 유발할 수 있습니다. 모든 날짜 관련 상수를 동일한 방식으로(예: 1-12월 사용 후 조정) 처리하여 일관성을 유지하는 것이 좋습니다.

Comment on lines 45 to +48
"안녕하세요," +
name +
"님\n" +
'2025년 2학기 에코노베이션 30기 신입회원 모집에 지원해주셔서 감사드립니다.\n한 번 제출하신 지원서는 수정이 불가능하며, 서류 합격 여부 및 면접 시간은 9월 12일 금요일에 안내될 예정입니다.\n더 궁금하신 내용은 카카오톡 채널 "에코노베이션"으로 문의 주시길 바랍니다.\n감사합니다.',
'2026년 1학기 에코노베이션 31기 신입회원 모집에 지원해주셔서 감사드립니다.\n한 번 제출하신 지원서는 수정이 불가능하며, 서류 합격 여부 및 면접 시간은 3월 14일 금요일에 안내될 예정입니다.\n더 궁금하신 내용은 카카오톡 채널 "에코노베이션"으로 문의 주시길 바랍니다.\n감사합니다.',

Choose a reason for hiding this comment

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

medium

템플릿 리터럴을 사용하면 여러 줄의 문자열과 변수를 더 쉽게 조합할 수 있어 가독성이 향상됩니다. 현재 문자열 연결(+) 방식 대신 템플릿 리터럴을 사용하는 것을 제안합니다.

      `안녕하세요,${name}님\n2026년 1학기 에코노베이션 31기 신입회원 모집에 지원해주셔서 감사드립니다.\n한 번 제출하신 지원서는 수정이 불가능하며, 서류 합격 여부 및 면접 시간은 3월 14일 금요일에 안내될 예정입니다.\n더 궁금하신 내용은 카카오톡 채널 "에코노베이션"으로 문의 주시길 바랍니다.\n감사합니다.`,

const ApplicationTexarea = ({ data }: ApplicationTextareaProps) => {
const textData = data as ApplicationTextarea;
const maxLength = ['deep', 'restoration', 'studyPlan'].includes(textData.name) ? 500 : 800;
const maxLength = ['deep', 'failure', 'studyPlan'].includes(textData.name) ? 500 : 800;

Choose a reason for hiding this comment

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

medium

하드코딩된 문자열 배열 ['deep', 'failure', 'studyPlan']과 숫자 500, 800은 '매직 스트링'과 '매직 넘버'에 해당하여 유지보수를 어렵게 할 수 있습니다. 이 값들을 파일 상단이나 별도의 상수 파일에 명확한 이름의 상수로 정의하여 사용하는 것을 권장합니다.

];

export const MAX_TEXT_LENGTH = 1000;
export const MAX_BOOLEAN_TEXT_LENGTH = 800;

Choose a reason for hiding this comment

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

medium

MAX_BOOLEAN_TEXT_LENGTH라는 이름은 BooleanTextarea 컴포넌트에만 국한되는 것처럼 보입니다. 하지만 Textarea.component.tsx에서도 동일한 값(800)을 하드코딩하여 사용하고 있습니다. 여러 곳에서 사용될 수 있는 값이라면 TEXT_LENGTH_LONG과 같이 좀 더 일반적인 이름으로 상수를 정의하는 것이 좋습니다. 또한, 500과 같은 다른 글자 수 제한도 TEXT_LENGTH_SHORT 등으로 함께 상수로 관리하면 코드의 일관성과 재사용성이 향상될 것입니다.

Suggested change
export const MAX_BOOLEAN_TEXT_LENGTH = 800;
export const TEXT_LENGTH_LONG = 800;

Copy link
Collaborator

@cho-in-sik cho-in-sik left a comment

Choose a reason for hiding this comment

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

수고하셨습니다-! 디자이너 10번 깊게 빠져본 경험 500자 적용이 안된 것 같아요!
이것만 확인 부탁드릴게요

@jskimcs jskimcs closed this Feb 27, 2026
@jskimcs jskimcs reopened this Feb 27, 2026
@jskimcs
Copy link
Collaborator Author

jskimcs commented Feb 27, 2026

수고하셨습니다-! 디자이너 10번 깊게 빠져본 경험 500자 적용이 안된 것 같아요! 이것만 확인 부탁드릴게요

저는 500자로 뜨긴 하는데 어느 부분이 안 된 것처럼 보이는 걸까요..?

@cho-in-sik
Copy link
Collaborator

엇 캐시문제엿나 500자 확인됐습니다 감사합니다 !

@cho-in-sik cho-in-sik merged commit de376d9 into main Feb 27, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 리팩터링 관련 이슈

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FE] refactor: QA 텍스트 수정

2 participants