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

docs(react): useBlockPromiseMultipleClick 훅의 비교 예시에 debounce 버튼 추가 완료 #103

Conversation

Sangminnn
Copy link
Collaborator

Overview

먼저 이전에 기여한 useBlockPromiseMultipleClick 훅 관련 문서의 내용을 작성해주셔서 감사합니다 :)
해당 문서를 확인하다보니 설명에는 debounce의 사용과 비교하는 내용이 있었는데, 실제 예시에도 debounce와의 차이를 확인할 수 있는 부분이 있으면 좋겠다 생각하여 간단하게 올려봅니다!

useBlockPromiseMultipleClick.mov

PR Checklist

  • [O] All tests pass.
  • [O] All type checks pass.
  • [O] I have read the Contributing Guide document.
    Contributing Guide

@Sangminnn Sangminnn requested a review from ssi02014 as a code owner May 5, 2024 09:59
Copy link

changeset-bot bot commented May 5, 2024

⚠️ No Changeset found

Latest commit: 1992c03

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ssi02014
Copy link
Contributor

ssi02014 commented May 5, 2024

@Sangminnn
안녕하세요! 우선 PR생성해주셔서 감사합니다!

제가 useBlockPromiseMultipleClick에대한 debounce 예제 추가 너무 좋은데요!

그런데 고민되는 부분이 useBlockPromiseMultipleClick 문서 내에 debounce에 대한 예제까지 포함되면 해당 문서에서 많은 정보를 제공한다는 느낌이 드는 것 같습니다..!🤔🤔

아니면 차라리 문서 내에 useDebounce에 대한 언급도 있으니 useDebounce 훅에 대한 실제 예제를 추가해서 해당 문서를 더 보강하는 것은 어떻게 생각하실까요!!?🙏🙄

@Sangminnn
Copy link
Collaborator Author

@ssi02014 좋은 의견 감사합니다! 말씀해주신 부분을 생각해보니 확실히 많은 정보를 제공하는 것으로 보이네요!
해당 풀리퀘는 닫고 useDebounce의 docs 보강에 대한 풀리퀘를 다시 드리도록 하겠습니다 :)

@Sangminnn Sangminnn closed this May 5, 2024
@Sangminnn Sangminnn deleted the docs/useBlockPromiseMultipleClick branch May 5, 2024 12:28
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