Feat/#264: 어학 공지사항 기능 추가 #297
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
어학과 채용 공지 UI 에 필요한 아이콘 추가
Primary color 를 조금 더 진한 파란색으로 변경
해당 컴포넌트에 취업 길라잡이, 어학 공지사항 카드 추가
해당 상수값이 다른 변수에 재할당되어 타입 에러가 발생하는 문제 해결을 위해 readonly 타입 제거
기존에는 학과가 있는지 없는지에 대한 값을 매개변수로 받았지만 확장성을 위해 엔드포인트를 직접 받아서 요청하도록 변경
hwinkr
reviewed
Dec 24, 2023
| link: string; | ||
| } | ||
|
|
||
| const InformHalfCardList = ({ |
There was a problem hiding this comment.
해당 컴포넌트는 리스트보다 하나의 카드를 보여주는 컴포넌트 같은데 List를 컴포넌트 이름에서 제거하는게 어떨까요~?
src/pages/Home/index.tsx
Outdated
Comment on lines
15
to
28
| <HalfCardWrapper> | ||
| <InformHalfCardList | ||
| iconKind="account" | ||
| title="취업 길라잡이" | ||
| subTitle="채용 정보 확인" | ||
| link="/announcement/recruit/normal" | ||
| /> | ||
| <InformHalfCardList | ||
| iconKind="language" | ||
| title="어학 공지사항" | ||
| subTitle="어학 정보 확인" | ||
| link="/announcement/language/normal" | ||
| /> | ||
| </HalfCardWrapper> |
There was a problem hiding this comment.
InformHalfCardList와 InformHalfCard를 구분해서 만들고,
InformHalfCardList 내부에서 InformHalfCard 을 호출하는건 어떨까요?
There was a problem hiding this comment.
그러면 InformHalfCardList -> InformHalfCard
HalfCardWrapper -> InformHalfCardList
가 되는건가요?
hwinkr
approved these changes
Dec 28, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🤠 개요
closes: Feat: 홈 화면에 어학 관련 카드 추가 #264
어학 공지사항, 채용 공지사항의 카드 컴포넌트를 추가했어요
어학 공지사항은 확인이 가능하지만 채용 공지사항은 아직 에러가 발생해요
기존에 공지사항을 서버에 요청하는 방식을 일부 수정했어요 (이전에는 학과가 있는지 없는지 매개변수로 넘겨줬다면 현재는 엔드포인트값을 직접 넘겨주도록 변경했어요)
나중에 작업할 채용 공지사항의 경우 업로드 날짜가 아닌 채용 기간을 보여주기에 해당 사항도 일부 수정할 필요가 있어보여요!
💫 설명
📷 스크린샷 (Optional)