부림이 0.4.3v 배포 #302
Merged
부림이 0.4.3v 배포 #302
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
- AnnouncementCategory : 'shool', 'major'와 같이 공지사항 분류 타입을 설정 - AnnouncementType : '일반', '고정' 처럼 확인 가능한 공지사항의 분류 타입을 설정
- 컴포넌트 내부에서 정의한 onClick 메서드 대신, 유틸 함수 openLink 함수 사용 - 경계선을 표현하는 스타일 컴포넌트 이름 변경 -> Vertical, Horizon 으로 종류를 구분
- prop으로 받는 type은 normal, pinned, search 셋 중 하나이며 type에 따라 다른 데이터를 사용해 렌더링 하도록 함 - search 같은 경우는 검색, 검색 결과를 확인, 검색 결과를 렌더링등 로직이 많기 때문에 AnnounceSearchList 컴포넌트를 추가로 구현
- 전공 컨텍스트에서 전공과 관련되어 있는 졸업요건 링크도 책임을 져야 한다고 판단
- TITLE: 공지사항 제목 - CATEGORY: 공지사항 분류, 어학 취업정보가 추가될 경우 여기에 추가할 수 있도록 함 - TYPE: API를 통해 가져온 공지사항을 분류
- 아직 상수로 빼지 못한 path들은 이 후 작업에서 빼도록 함
- announcement/<category>/<type> 에 따라서 컴포넌트를 렌더링 하도록 함
- title : 공지사항 제목을 표현 ex) 학교 공지사항 - category : 공지사항 분류를 표현 ex) announcement/school -> 학교 공지사항 - endPoint : 공지사항을 가져오기 위한 api endPoint를 표현 - 이 3가지 props만 전달하면 검색, 일반 & 공지 필터링, 공지사항 리스트 렌더링을 책임지는 컴포넌트를 렌더링 할 수 있음
- InformCardList: 공지사항 관련 카드 컴포넌트를 추상화 - 홈페이지 컴포넌트에서 담당하던 graduationLink의 상태 관리를 MajorPovider로 이전
- 커스텀 훅 사용 - 버튼의 자식 요소였던 span 제거
- Map(Provider) 컴포넌트만 의존해서 렌더링할 수 있음 - Provider에서 의존성을 주입하고, 커스텀 훅으로 로직을 꺼내서 사용하기 때문에 의존성 주입을 외부에 드러내지 않아도 됨
어학과 채용 공지 UI 에 필요한 아이콘 추가
Primary color 를 조금 더 진한 파란색으로 변경
해당 컴포넌트에 취업 길라잡이, 어학 공지사항 카드 추가
해당 상수값이 다른 변수에 재할당되어 타입 에러가 발생하는 문제 해결을 위해 readonly 타입 제거
기존에는 학과가 있는지 없는지에 대한 값을 매개변수로 받았지만 확장성을 위해 엔드포인트를 직접 받아서 요청하도록 변경
- 모달 합성 컴포넌트로 변경 필요
Refactor/#293: 모달 합성 컴포넌트 적용 && 지도 페이지 의존성 주입 Context API 적용
feat/#294 : 건의사항 페이지 구현
Feat/#264: 어학 공지사항 기능 추가
- 건의사항을 보낼 때 한번 더 내용을 확인하는 로직 추가
chore/#300 : 건의사항 페이지에서 사용하는 모달을 합성 컴포넌트로 변경
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.
🤠 개요
💫 설명
Modal,UpperLayout)📷 스크린샷 (Optional)