morethanmin / morethan-log

😎 A static blog using notion database
https://morethan-log.vercel.app
MIT License
1.95k stars 1.57k forks source link

add useBooleanState custom hook #276

Closed SEOKKAMONI closed 1 year ago

SEOKKAMONI commented 1 year ago

Description

use useBooleanState

영어를 잘 못해서.. 한국어로 써야 하는 이유를 간단하게 작성하겠습니다! 우선 useBooleanState를 사용한 이유는 유지 보수성도 있지만 보다 명시적인 네이밍입니다

기존에 useState만 사용하고 있다면 한 번 더 추상화를 해줘야 하지만 useBooleanState를 사용한다면 한 번 더 불필요한 추상화가 필요 없습니다 또한 useState보다 다양한 기능을 지원하므로 여러 동작을 할 수 있는 것도 강점이라고 생각합니다!

AS-IS

const [isOpen, setIsOpen] = useState(false);

const openDropdown = () => {
   setIsOpen(true);
}

TO-BE

const { setTrue: openDropdown } = useBooleanState();

하지만 현재 booleanState는 하나 밖에 사용되지 않으니 메인 테이너님께서 판단 후 리뷰 부탁드립니다!! 좋은 블로그 만들어주셔서 감사합니다 :)

Related tickets

https://github.com/morethanmin/morethan-log/issues/XX

PR Checklist

vercel[bot] commented 1 year ago

Someone is attempting to deploy a commit to a Personal Account owned by @morethanmin on Vercel.

@morethanmin first needs to authorize it.

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
morethan-log ❌ Failed (Inspect) Sep 3, 2023 10:08am
SEOKKAMONI commented 1 year ago

@morethanmin 바쁘신데 정말 수고많으십니다 :) 해당 PR에 대해서 어떻게 생각하시는지 개발자님에 생각이 궁금합니다