Skip to content

feat(web): 메뉴 아래 프로필을 눌러 누구로 볼지 이슈 담당자 중에서 고름 (SIL-74) - #62

Merged
space-angel merged 1 commit into
mainfrom
feat/sil-74-profile-me-picker
Oct 16, 2026
Merged

space-angel merged 1 commit into
mainfrom
feat/sil-74-profile-me-picker

Conversation

@space-angel

Copy link
Copy Markdown
Contributor

관련 이슈

SIL-74 나에게 온 변경 화면 (웹 앱 배포 전 준비, 배포는 SIL-84)

무엇을 바꿨나요

  • 왼쪽 메뉴 아래 프로필을 누르면 "누구로 볼까요?" 목록이 위로 열려요. 이름을 고르면 그 사람에게 온 변경을 다시 불러와요. 지금 이름에는 "지금" 표시가 붙어요.
  • 목록은 이슈 담당자 이름이에요 (api.listPeople(), 겹치지 않게 가나다순). 확인할 변경은 이슈 담당자에게 가서, 담당자가 아닌 이름으로는 받을 변경이 없어요.
  • 고른 이름은 이 브라우저가 기억해요 (me.ts의 saveMe). 주소에 ?me=이름을 붙여 여는 방법도 그대로 돼요.
  • 이름이 없을 때 안내와 "받은 사람만 답할 수 있어요" 안내를 ?me= 대신 "왼쪽 아래 프로필"로 바꿨어요.
  • 사람을 바꾸면 고른 변경·보기 범위는 처음부터 다시 시작해요.
  • README와 docs/타입명세서.md의 SilApi·내 이름 설명을 고쳤어요.

왜 바꿨나요

로그인 기능이 없어서, 배포한 뒤 팀원이 주소에 ?me=를 직접 붙이지 않고도 자기 이름으로 들어갈 수 있게 하려고요 (10/16 호윤).

어떻게 확인했나요

  • pnpm typecheck, pnpm lint, pnpm format:check 통과, 웹·packages 테스트 75개 통과
  • pnpm app:web(예시 데이터)로 열어서 확인: 프로필 누르면 목록이 열림 → ?me=호윤으로 연 뒤 민수를 고르면 민수 목록으로 바뀌고 브라우저에 저장됨 → Esc·바깥 클릭으로 닫힘

직접 확인하지 못한 부분

  • 진짜 DB(Supabase)로 연 화면은 확인하지 못했어요. 예시 데이터에는 담당자가 민수 한 명뿐이라 목록에 한 명만 보였어요.
  • apps/detect의 t1 테스트 11개는 이 컴퓨터 셸의 Node가 22라서 실패했어요(.ts 직접 실행 불가). 이번 변경과 상관없고 CI(Node 24)에서 확인해요.

🤖 Generated with Claude Code

로그인이 없어서 주소에 ?me=를 붙여야만 사람을 바꿀 수 있었어요. 프로필을 누르면
이슈 담당자 이름 목록(api.listPeople)이 뜨고, 고르면 그 사람의 변경을 다시 불러와요.
고른 이름은 브라우저가 기억하고, ?me=로 여는 방법도 그대로 둬요.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@space-angel
space-angel merged commit c5bedf9 into main Oct 16, 2026
@github-actions

Copy link
Copy Markdown

바뀐 결정 사항 찾기 · PR #62

지금은 결정 사항을 저장하지 않고 결과만 알려 주는 기간이에요. 저장했다면 아래처럼 했을 거예요.

@space-angel 이 PR에서 결정 사항이 바뀐 것 같은데 확실하지 않아요.

원래 '이름을 고르는 화면은 만들지 않는다'고 정했던 것을 '왼쪽 메뉴 아래 프로필을 눌러 이슈 담당자 중에서 내 이름을 고른다'로 바꾼 것인데, 이 내용을 새 결정 사항으로 남길까요?

결정 사항은 그대로 두었어요. 바뀐 게 맞으면 Claude Code에서 "결정 사항을 바꿔 줘"라고 말해 주세요. 바뀐 게 아니면 이 댓글은 넘어가셔도 돼요.

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.

1 participant