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/#287] 참여자 가능시간 입력시 새로고침하면 참여자 이름이 날아가는 문제 수정 #289

Merged
merged 2 commits into from
Jul 30, 2024

Conversation

simeunseo
Copy link
Member

@simeunseo simeunseo commented Jul 27, 2024

🌀 해당 이슈 번호

🔹 어떤 것을 변경했나요?

  • recoil-persist로 참여자 이름 저장

🔹 어떻게 구현했나요?

참여자 이름을 저장하는 atom에 recoil-persist를 적용했습니다

export const userNameAtom = atom<string>({
  key: 'userNameAtom',
  default: '',
  effects_UNSTABLE: [persistAtom],
});

recoil-persist는 해당 정보를 local-storage에 저장해주는 기능입니다.
그래서 참여자 이름을 입력하면 다음과같이 loca-storage에 저장되고
새로 입력할때마다 값이 대체됩니다.
image

🔹 PR 포인트를 알려주세요!

일단 빠르고 간단한 해결책인 recoil-persist로 해결을 하긴했지만
전역 상태관리에 대해 전반적으로 리팩토링이 필요할것같습니다.

🔹 스크린샷을 남겨주세요!

수정 전

2024-07-27.11.32.01.mov

수정 후

2024-07-27.11.31.13.mov

@simeunseo simeunseo changed the title feat: recoil-persist 적용 [Feat/#287] recoil-persist 적용 Jul 27, 2024
@simeunseo simeunseo requested a review from ljh0608 July 27, 2024 14:32
@simeunseo simeunseo self-assigned this Jul 27, 2024
@simeunseo simeunseo changed the title [Feat/#287] recoil-persist 적용 [Feat/#287] 참여자 가능시간 입력시 새로고침하면 참여자 이름이 날아가는 문제 수정 Jul 27, 2024
@simeunseo simeunseo added the fix fix label Jul 27, 2024
Copy link
Member

@ljh0608 ljh0608 left a comment

Choose a reason for hiding this comment

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

LGTM

@simeunseo simeunseo merged commit 512b9eb into develop Jul 30, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 참여자 가능시간 입력시 새로고침하면 참여자 이름이 날아가는 문제 수정
2 participants