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.

Sign up for GitHub

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

Jump to bottom

[오혜성] 5장: 리액트와 상태 관리 라이브러리 #29

Merged
merged 2 commits into from
Jan 9, 2024
Merged

[오혜성] 5장: 리액트와 상태 관리 라이브러리 #29

merged 2 commits into from
Jan 9, 2024

Conversation

hyesungoh
Copy link
Member

:goberserk:

@hyesungoh hyesungoh self-assigned this Dec 20, 2023
@github-actions github-actions bot added the 5장 label Dec 20, 2023
@Dongkyuuuu
Copy link
Contributor

멈춰....

@hyesungoh hyesungoh enabled auto-merge (squash) January 2, 2024 13:43
eunddodi
eunddodi approved these changes Jan 8, 2024
View reviewed changes
Copy link
Contributor

@eunddodi eunddodi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏👏👏👏👏

Seojunhwan
Seojunhwan approved these changes Jan 8, 2024
View reviewed changes
Copy link
Member

@Seojunhwan Seojunhwan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨어요!

5장/오혜성.md
Comment on lines +48 to +50
```
하나의 이슈에 대해 여러가지 해결책이 나온다는 것은 그만큼 이 분야가 건강하게 성장하고 있다는 증거이기도 하다
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏿👍🏿

5장/오혜성.md
+ Recoil과는 다르게, 컴포넌트 루트 레벨에서 Context가 존재하지 않아도 됨
- Context가 없다면 Provider가 없는 형태로 기본 스토어를 루트에 생성하고 이를 활용해 값을 저장하기 때문
+ store에 atom 객체 그 자체를 키로 활용해 값을 저장 `WeakMap`
- 🤘 framer motion에서는 성능을 위해 해당 자료형을 사용
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오옹

Dongkyuuuu
Dongkyuuuu approved these changes Jan 9, 2024
View reviewed changes
Copy link
Contributor

@Dongkyuuuu Dongkyuuuu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍👍👍👍👍👍👍👍👍👍

5장/오혜성.md

```
- https://react-ko.dev/reference/react/useSyncExternalStore
- https://junghyeonsu.com/posts/react-use-sync-external-store/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍👍👍

5장/오혜성.md
+ 메타 팀에서 주도적으로 개발하고 있기 때문에 다른 라이브러리보다 잘 지원할 것으로 기대됨

```
axios도 드디어 1.0.0을 릴리스
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

어느새 1.6까지 왔네요...

@hyesungoh hyesungoh merged commit 9216909 into main Jan 9, 2024
1 check passed
@hyesungoh hyesungoh deleted the 5장/오혜성 branch January 9, 2024 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eunddodi eunddodi eunddodi approved these changes

@Seojunhwan Seojunhwan Seojunhwan approved these changes

@Dongkyuuuu Dongkyuuuu Dongkyuuuu approved these changes

@oyeon-kwon oyeon-kwon Awaiting requested review from oyeon-kwon oyeon-kwon is a code owner

@sensecodevalue sensecodevalue Awaiting requested review from sensecodevalue sensecodevalue is a code owner

@sjoleee sjoleee Awaiting requested review from sjoleee sjoleee is a code owner

@WooWan WooWan Awaiting requested review from WooWan WooWan is a code owner

Assignees

@hyesungoh hyesungoh

Labels
5장
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@hyesungoh @Dongkyuuuu @Seojunhwan @eunddodi