Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor: 구 관심사 & 직무 정보 선택 UI 리팩터링 #248

Merged
merged 4 commits into from
Dec 20, 2023

Conversation

sscoderati
Copy link
Member

@sscoderati sscoderati commented Dec 19, 2023

이슈번호

close: #240

작업 내용 설명

Dec-19-2023 21-19-14

기존의 관심사 및 직무 정보를 선택하는 선택 UI를 개편했습니다.
기존 UI는 목록을 초과해서 선택하면 에러 메시지를 띄우는 기능이 있었는데, 해당 기능을 React Hook Form에서 제공하고 있어서 제거했습니다. (추후 zod와 함께 적용 예정)
원래 이슈에는 JobStore, InterestStore를 제거하는 것까지 태스크로 잡아놨었는데, 폼 개선 작업 때 이루어져야 할 것 같습니다..!
React Hook Form의 Controller를 활용 시 아래와 같은 방식으로 활용하면 됩니다.

// 활용 예시

// ...
interface MultiSelectorSubmitValue {
  interest: string[];
}

const TestPage = () => {
  const { control, handleSubmit } = useForm<MultiSelectorSubmitValue>();
  const handleSubmitInterests: SubmitHandler<MultiSelectorSubmitValue> = (data) => {
    console.log(data);
  };
  return (
    <TestPageLayout>
      <form onSubmit={handleSubmit(handleSubmitInterests)}>
        <Controller
          name={"interest"}
          control={control}
          render={({ field }) => (
            <MultiSelector
              isDarkMode={false}
              itemList={InterestList}
              maxCount={4}
              onValueChange={field.onChange}
            />
          )}
        />
        <button type={"submit"}>{"Submit Interest"}</button>
      </form>
    </TestPageLayout>
  );
};

// ...

리뷰어에게 한마디

이 PR을 머지하고 기존 폼 개선 작업이 이루어져야 합니다...!

@sscoderati sscoderati added the 🧹 Refactor 코드 구조 개선 & 가독성 향상 label Dec 19, 2023
@sscoderati sscoderati requested a review from judahhh December 19, 2023 12:40
@sscoderati sscoderati self-assigned this Dec 19, 2023
@sscoderati sscoderati changed the title Refactor: 구 MultiSelector 리팩토링 Refactor: 구 관심사 & 직무 정보 선택 UI 리팩터링 Dec 19, 2023
@judahhh judahhh merged commit 679c5f8 into dev Dec 20, 2023
1 check passed
@judahhh judahhh deleted the refactor/renovate-selector branch December 20, 2023 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧹 Refactor 코드 구조 개선 & 가독성 향상
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

🔨 [Refactor] 관심사, 직무 Selector 리팩터링 및 개선 작업
2 participants