Skip to content

fix: 공개 범위 설정 저장 문제 해결#162

Merged
deholic merged 1 commit intomainfrom
feature/fix-visibility-save-issue
Jan 14, 2026
Merged

fix: 공개 범위 설정 저장 문제 해결#162
deholic merged 1 commit intomainfrom
feature/fix-visibility-save-issue

Conversation

@deholic
Copy link
Copy Markdown
Owner

@deholic deholic commented Jan 14, 2026

요약

  • ComposeBox 컴포넌트에서 공개 범위 설정 변경시 저장되지 않는 문제 해결
  • useEffect 간의 레이스 컨디션 제거로 안정적인 상태 관리 구현

문제

  • 사용자가 "전체 공개"에서 "미등록"으로 공개 범위를 변경할 때 localStorage에 저장되지 않음
  • 두 개의 useEffect가 서로 충돌하여 상태 덮어쓰기 발생

해결

  • visibilityvisibilityAccountId를 통합한 visibilityState 객체로 상태 관리
  • 계정 변경시에만 상태를 다시 로드하도록 로직 최적화
  • 단일 useEffect로 저장 로직 단순화

변경 내용

  • 상태 관리: 분리된 상태 → 통합된 객체
  • 저장 로직: 조건부 저장 → 명확한 저장 조건
  • 계정 변경: 항상 재로드 → 필요시에만 재로드

테스트

  • 모든 공개 범위(전체 공개, 미등록, 팔로워, DM) 변경시 정상 저장 확인
  • 계정 전환 시 각 계정의 공개 범위 설정 독립적으로 동작 확인
  • TypeScript 타입 체크 및 빌드 성공 확인

- useEffect 간의 레이스 컨디션 제거
- visibility와 visibilityAccountId 상태를 통합하여 visibilityState 객체로 관리
- 계정 변경시에만 상태를 다시 로드하도록 최적화
- "전체 공개"에서 "미등록"으로 변경시 저장되지 않던 문제 해결
@deholic deholic self-assigned this Jan 14, 2026
@deholic deholic added the enhancement New feature or request label Jan 14, 2026
@deholic deholic added this to the v0.11.0 milestone Jan 14, 2026
@deholic deholic merged commit ea70c79 into main Jan 14, 2026
1 check passed
@deholic deholic deleted the feature/fix-visibility-save-issue branch January 14, 2026 04:11
@deholic deholic restored the feature/fix-visibility-save-issue branch January 15, 2026 02:52
@deholic deholic deleted the feature/fix-visibility-save-issue branch January 15, 2026 02:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant