feat: 29기 지원서 질문 데이터 추가 #249

jiucchu · 2025-02-27T21:35:58Z

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

29기 신입모집을 위한 지원서 질문을 추가하고, 관련 페이지를 업데이트했습니다.

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

1. 필수 확인 사항 분리

개인정보수집과 함께 묶여있던 확인 문구를 개별 항목으로 분리하고, 문구를 수정했습니다.

2. 공통 질문 사항 수정

공통 질문 사항 중, 3. 기본 인적 사항을 3. 전공을 입력해주세요. 로 변경했습니다.

3. 개발자 질문 8번 타입 변경

기존 BooleanTextarea 이었던 질문을 수정하고, Horizontal 타입으로 변경했습니다.

4. 기타 요구사항에 맞춘 질문 수정

분야별 질문 중 '실패 경험' 질문과 '무언가에 열정을 쏟은 경험' 순서 교체

진로에 관련한 분야별 질문 수정
본인이 생각하는 진로와 이를 위해 노력한 점에 대해 이야기해주세요. -> {특정 분야}를 희망하는 이유는 무엇인가요?

면접 시간 및 개인정보 상수 데이터 업데이트

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

1. applicationLayout의 줄바꿈 속성 변경

기존 break-keep(단어 단위 줄바꿈)으로 적용되어 있던 속성을 break-all(문자 단위 줄바꿈)으로 변경했습니다.

2. Horizontal.componet의 subtitle 코드 리팩토링

subtitle 내부에서 볼드 태그를 사용하기 위해 코드를 리팩토링 했습니다.

3. ApplicationNavbar의 button 스타일 수정

navbar에 내용을 추가 한뒤 16인치 이상의 모니터에서 아래와 같이 뒤틀리는 모습을 발견했습니다.
이를 해결하기 위해 Navbar의 button에 w-full 속성을 추가했습니다.

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

이전 기수보다 지원지 질문 변경이 많은 편입니다. 반영되지 않은 사항 혹은 오탈자가 있는지 확인 부탁드립니다.
기존 한 페이지에 묶여있던 질문을 따로 분리했습니다. (필수 확인 사항) 이로 인해 발생하는 사이드 이펙트가 있을지 중심적으로 봐주시면 감사하겠습니다!

체크리스트 ✅

reviewers 설정
assignees 설정
label 설정

cho-in-sik

고생하셨습니다 👍

cho-in-sik · 2025-02-28T03:29:21Z

frontend/src/constants/applicant/29.ts

+    title: "기본 인적 사항을 입력해주세요.",
+    type: "customHuman",
+    value: {
+      hunamName: { name: "name" },


hunamName -> humanName 오탈자 있습니다!
근데 전 기수 질문지에도 hunamName이어서 바꾼다면 타입까지 바꿔주시면 감사하겠습니다~

smb0123

alert 창 메시지와 input에 입력해야 하는 메시지가 다른데 질문의 멘트 수정 (확인했습니다 => 확인했습니다.)과 alert 창 메시지 변경 (체크해주세요. => 입력해주세요.)로 변경하는 건 어떤가요?

geongyu09

고생 많으셨습니다!
놓친 부분이 있는지를 중점적으로 보려고 했는데, 문제 없어보입니다!

geongyu09 · 2025-02-28T04:43:38Z

frontend/app/application/page.tsx

 import {
  APPLICATION_MANAGER,
  APPLICATION_NAVBAR_MANAGER,
-} from "@/src/constants/application/28/manager";
+} from "@/src/constants/application/29/manager";


해당 부분도 나중에는 동적 임폴트문으로 변경해도 좋을 것 같아요!
하나씩 변경하는 것은 아무래도 실수를 범할 위험성이 있는 것 같습니다

언젠간 서버로 넘겨야할 놈들이네요..

사실 저렇게 하지 않고, url에 기수를 집어 넣으므로서 작동하게 하고 싶었습니다...
하지만 BE에서는 허락해주지 않는 그런 컨택스트가 있습니다.

smb0123 · 2025-02-28T06:37:53Z

frontend/components/application/applicationLayout/Horizontal.componet.tsx

-            <Txt className="text-sm whitespace-pre-wrap">
-              {applicationQuestion.subtitle}
-            </Txt>
+            <>


Fragment 태그를 빼도 될 것 같습니다.

smb0123 · 2025-02-28T07:02:30Z

frontend/src/constants/application/29/designer.ts

+    title:
+      "에코노베이션은 3학기 이상의 활동과 매주 금요일 17시 주간발표 참여가 필수이며, 최종 합격 시 신입 기수로 구성된 팀으로 한 학기 동안 활동하게 됩니다. \n또한, 에코노베이션에서 운영되고 있는 다양한 부서 활동과 네트워킹 행사에 적극적인 참여를 권장합니다.",
+    subtitle:
+      "위 내용을 확인하셨으면 <b>'확인했습니다'</b>를 기입해주세요.\n확인하지 않았을 시 합격이 취소될 수 있습니다.",


확인했습니다 => 확인했습니다.
변경부탁드립니다.

개발자와 기획자 파일도 수정 부탁드려요 !

수정 완료 했습니다!

smb0123

고생하셨습니다 ~

2yunseong

PR 메세지 열심히 작성해주셔서, 코드를 이해하는 데 편했습니다 :)
첫 기여 축하드립니다 👍
또 가장 하드하게 코딩된 부분 작업이라, 더더욱 고생하셨습니다 😭

허나, git add . 하시는 습관은 안좋은 습관입니다 🥲 한 가지만 확인 부탁드립니다 ✅

2yunseong · 2025-02-28T13:09:22Z

frontend/app/application/page.tsx

 import {
  APPLICATION_MANAGER,
  APPLICATION_NAVBAR_MANAGER,
-} from "@/src/constants/application/28/manager";
+} from "@/src/constants/application/29/manager";


언젠간 서버로 넘겨야할 놈들이네요..

2yunseong · 2025-02-28T13:10:22Z

frontend/components/applicant/applicantNode/CustomHuman.component.tsx

@@ -17,7 +17,7 @@ const ApplicantCustomHuman = ({
    <>
      <div className="flex items-center pt-6 pb-12">
        <Txt typography="h1" className="px-6 min-w-fit">
-          {applicantDataFinder(data, customHumanData.value.hunamName.name)}
+          {applicantDataFinder(data, customHumanData.value.humanName.name)}


이 방식(applicantDataFinder)의 진짜 큰 문제점.. 휴먼에러를 못 잡습니다 ㅠ

2yunseong · 2025-02-28T13:13:35Z

frontend/package-lock.json

의존성 변경이 없는것 같은데 package-lock까지 커밋한 이유가 있을까요?? 추가로, 우리 팀은 pnpm을 사용하여야 합니다.
불필요한 lock 파일이 올라와 있는 것 같아요.

2yunseong · 2025-02-28T13:14:53Z

package-lock.json

이 파일이 커밋된 이유를 설명해주실 수 있나용??

jiucchu requested review from 2yunseong, geongyu09, cho-in-sik and smb0123 February 27, 2025 21:35

jiucchu self-assigned this Feb 27, 2025

jiucchu requested review from BlackBean99 and loopy-lim as code owners February 27, 2025 21:35

jiucchu added the feature 기능개발 label Feb 28, 2025

cho-in-sik reviewed Feb 28, 2025

View reviewed changes

cho-in-sik approved these changes Feb 28, 2025

View reviewed changes

jiu added 10 commits February 28, 2025 14:51

refactor: 줄바꿈 설정 변경

f5318a1

chore: 지원서 페이지 분야별 질문 29기로 변경

7c0f19a

refactor: 줄바꿈을 위한 속성 추가

4313252

refactor: subtitle bold 처리를 위한 리팩토링

e95cef1

chore: 29기 지원서 공통 질문 추가

20c4fa7

chore: 29기 디자이너 분야별 질문 추가

5657cb5

chore: 29기 개발자 분야별 질문 추가

43b4ebc

chore: 29기 지원서 기획자 분야별 질문 추가

041b83f

chore: 면접 가능 시간 업데이트

07f2ed6

chore: 기획자 14번 질문 id 수정

b37e90e

smb0123 reviewed Feb 28, 2025

View reviewed changes

jiu added 4 commits February 28, 2025 15:00

chore: 29기 지원서 추가

be300ed

chore: 29기 지원서 평가 기준 업데이트

ec9da34

chore: 지원서 기수 업데이트

1bd72f7

fix: ApplicationNavbar가 올바르게 보이지 않는 문제 해결

8ea2e35

jiucchu force-pushed the feat/241-update-application branch from df17b76 to 8ea2e35 Compare February 28, 2025 06:07

jiu added 2 commits February 28, 2025 15:18

refactor: humanName 오타 수정

fbedd5e

refactor: 개인 정보 수집 동의 alter 문구 수정

40ec4a1

geongyu09 approved these changes Feb 28, 2025

View reviewed changes

refactor: CustomHuman 컴포넌트 humanName 오타 수정

aec213b

smb0123 reviewed Feb 28, 2025

View reviewed changes

smb0123 approved these changes Feb 28, 2025

View reviewed changes

jiu added 2 commits February 28, 2025 16:15

refactor: 필수 확인 문구 온점 추가

75b75fc

refactor: Fregment 태그 제거

5f445d1

2yunseong requested changes Feb 28, 2025

View reviewed changes

chore: 불필요한 의존성 파일 삭제

a488b50

jiucchu force-pushed the feat/241-update-application branch from 635dc5c to a488b50 Compare March 1, 2025 01:35

jiucchu requested a review from 2yunseong March 1, 2025 05:48

2yunseong approved these changes Mar 1, 2025

View reviewed changes

jiucchu merged commit 0f6d00b into main Mar 1, 2025
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: 29기 지원서 질문 데이터 추가 #249

feat: 29기 지원서 질문 데이터 추가 #249

jiucchu commented Feb 27, 2025 •

edited

Loading

cho-in-sik left a comment

cho-in-sik Feb 28, 2025

smb0123 left a comment

geongyu09 left a comment

geongyu09 Feb 28, 2025

2yunseong Feb 28, 2025

loopy-lim Mar 1, 2025

smb0123 Feb 28, 2025 •

edited

Loading

smb0123 Feb 28, 2025

smb0123 Feb 28, 2025

jiucchu Feb 28, 2025

smb0123 left a comment

2yunseong left a comment •

edited

Loading

2yunseong Feb 28, 2025

2yunseong Feb 28, 2025

2yunseong Feb 28, 2025

2yunseong Feb 28, 2025

feat: 29기 지원서 질문 데이터 추가 #249

feat: 29기 지원서 질문 데이터 추가 #249

Conversation

jiucchu commented Feb 27, 2025 • edited Loading

관련 이슈

작업 분류

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

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

1. 필수 확인 사항 분리

2. 공통 질문 사항 수정

3. 개발자 질문 8번 타입 변경

4. 기타 요구사항에 맞춘 질문 수정

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

1. applicationLayout의 줄바꿈 속성 변경

2. Horizontal.componet의 subtitle 코드 리팩토링

3. ApplicationNavbar의 button 스타일 수정

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

체크리스트 ✅

cho-in-sik left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

smb0123 left a comment

Choose a reason for hiding this comment

geongyu09 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

smb0123 Feb 28, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

smb0123 left a comment

Choose a reason for hiding this comment

2yunseong left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jiucchu commented Feb 27, 2025 •

edited

Loading

smb0123 Feb 28, 2025 •

edited

Loading

2yunseong left a comment •

edited

Loading