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

고정된 Text값들을 공통으로 사용할 수 있도록 리팩토링 #94

Open
wooooooood opened this issue Jun 13, 2021 · 0 comments

Comments

@wooooooood
Copy link
Member

Is your feature request related to a problem? Please describe.

산업 리스트 / 직무 리스트 등은 공통적으로 사용되며 일관성이 유지되어야하는 상수값이라고 생각되는데, 지금은 이 값을 사용하는 컴포넌트/페이지에 각각 흩어져있습니다.

Describe the solution you'd like

예를 들면 SignuPage, StudyMakeModal, IndustrySelectModal에서 industry List를 동일하게 사용중입니다.
따라서 이러한 값들, 혹은 공통으로 사용할 수 있는 에러 메세지 등은 constants 파일에 분리하도록 리팩토링하면 추후에 목록 값을 확인하거나 수정하기에도 용이하리라 생각됩니다.

Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered.

Additional context

아래 두 리스트를 Constant로 분리. 파일명은 뭘로할지 생각해보기?

const jobList = [
  '경영/사무',
  '마케팅/MD',
  '영업',
  'IT/인터넷',
  '연구개발/설계',
  '생산/품질',
  '디자인',
  '기타',
];

const industryList = [
  '금융/은행',
  'IT',
  '서비스/교육',
  '보건/의약/바이오',
  '제조',
  '건설',
  '예술/문화',
  '기타',
];
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant