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: document #268

Merged
merged 11 commits into from
Nov 22, 2024
Merged

feat: document #268

merged 11 commits into from
Nov 22, 2024

Conversation

rabyeoljji
Copy link
Contributor

@rabyeoljji rabyeoljji commented Nov 22, 2024

개요

  • 노트 이동을 위한 체크리스트 훅 적용 (이번 버전에서 ignoreIds가 사용될지 모르겠어서 뺀 훅을 use-check-list로 둠)
  • 기존에 ignoreIds와 함께 사용할 수 있는 훅은 use-check-list-ignore-ids.ts로 이름을 변경해 남겨두었습니다.
  • directory context 적용해 id로 documents 가져오도록 적용
  • 기존에 잘못되어 있던 이름을 수정했습니다. DirectoryContext => DocumentContext (정우님이 만드신 directory context는 별도)
  • document쪽은 미리 queries를 적용

  • 스토리북에서 page 부분이 query client가 없어 오류가 발생하는 것을 발견했습니다.
  • 우선 client layout을 만들어 스토리북에서 감싸주었는데 이게 맞는 방법일지는 고민이 필요할 것 같습니다

세부 내용

관련 링크

@rabyeoljji rabyeoljji merged commit e58ad7d into develop Nov 22, 2024
5 checks passed
rabyeoljji added a commit to rabyeoljji/pick-toss-next that referenced this pull request Nov 22, 2024
* feat: document card check

* feat: query client for storybook

* fix: merge and fix error

* feat: directory id에 따른 documents 가져오기 + queries 적용

* fix: 불필요한 코드 삭제

* fix: build error

* fix: import

* fix: storybook

* fix: expend buttons bug
@rabyeoljji rabyeoljji self-assigned this Nov 26, 2024
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.

1 participant