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

Feat/naver book #147

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from
Open

Feat/naver book #147

wants to merge 5 commits into from

Conversation

Hys-Lee
Copy link
Collaborator

@Hys-Lee Hys-Lee commented Feb 4, 2025

동작 테스트 PR

검색에 대한 정렬 기능 때문에, 검색 기능만 제외함.

leochoi98 and others added 5 commits August 6, 2024 14:57
* init: 환경설정

* feat: notification 제작중

* feat: notification 헤더 추가

* feat: notification API 연결 및 type 따른 처리

* feat: notification css 수정

* fix: notification 타입 및 id 필드 변경에 따른 방어코드로 수정

* fix: notification api 함수 변경

* fix: notification 오브젝트 속성 참조 오류 수정

* fix: notification 드롭다운 아이템 key값 수정

* fix: notification 쿼리 키 설정 및 적용

* fix: notification antd-style적용, suspense제거.

* fix: notification 알림 문구 변경, 아이콘 변경

* fix: unread notification count와 알림 클릭 시 post로 읽음 처리 적용

post후 unread응답 다시 불러오기 적용
불필요한 옵셔널 체이닝 제거

* feat: notification clear기능 추가 및 ui변경

Co-authored-by: SangHyeon Lee <[email protected]>
fix: 메타태그 이미지 수정
@Hys-Lee Hys-Lee self-assigned this Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants