Skip to content

fix(web): 나에게 온 변경 화면을 신지님 시안(10/15)에 맞춤 (SIL-74) - #60

Merged
space-angel merged 9 commits into
mainfrom
feat/sil-74-match-prototype-1015
Oct 15, 2026
Merged

space-angel merged 9 commits into
mainfrom
feat/sil-74-match-prototype-1015

Conversation

@space-angel

@space-angel space-angel commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

관련 이슈

SIL-74 [개발] 웹 앱 화면 만들기 (A1~A4) · 시안 https://suju0.github.io/haribo-web/ (SIL-69)

무엇을 바꿨나요

  • 진행 단계: 새 변경 → 확인 완료 → 작업 반영 을 결정 사항 변경 → 확인 완료 → 이슈 반영으로. "작업 반영했어요" → "이슈에 반영했어요", [작업 반영] 칩 → [이슈 반영], 알림 "상태를 바꿨어요 · 이슈 반영"
  • 출처 칩: 칩 하나만 붙여요. "…에서 바꿈"을 빼고 [AI 대화] [코드] [Slack] [Figma] 등, AI가 찾았고 출처가 코드면 [PR에서 찾은 변경]. 따로 붙던 [PR에서 찾음]은 없앴어요. 모두 같은 모양이고 AI 대화에만 반짝 아이콘
  • 다시 반영 안내(목록·상세): "지난번 변경을 반영하셨다면 다시 고쳐야 해요" → "이전 결정 사항으로 작업 중이었다면 최신 결정 사항을 반영해 주세요"
  • 대체됨 상세: 위쪽 회색 상자 대신 "지금 해야 할 일" 옆 노란 안내 "확인할 변경이 새 버전으로 대체되었어요. 최신 버전의 확인할 변경에 답해 주세요." "새 버전 보러 가기"는 새 버전 카드가 있을 때만. 새 버전이 자동으로 닫혀 답할 카드가 없으면 안내는 목록과 같은 "새 버전으로 대체됐다가, 내가 이미 확인한 값으로 돌아왔어요. 답하지 않아도 돼요"
  • 안내 상자 색: 연한 주황·주황 글자 → 노랑(#FFF4C2)·검정 글자. 긴 안내가 줄바꿈되게 SectionTitle 오른쪽 칸 폭을 62%까지로
  • docs/타입명세서.md, apps/web/README.md 이름 함께 고침

상세 위 칩 (새 결정 사항 app.detail.change-letter)

  • 이슈로 묶어 "이슈 feat(notify): 기준이 바뀌면 알릴 담당자를 고르는 함수 추가 (SIL-67) #7" 칩 하나에 다른 결정 사항이 섞이던 것을, 결정 사항과 내 이슈 한 쌍마다 칩 하나로 나눔
  • 칩 이름은 받은 순서대로 변경 A, 변경 B… (결정 사항 ID는 너무 길고 정해진 번호가 없어서). 답하거나 새 변경이 와도 이미 붙은 글자는 그대로, 마우스를 올리면 결정 사항 제목 · 이슈 · 상태
  • 위키 앱-화면.md에 새 결정 사항을 더하고 INDEX.md 기준 수 11 → 12

문구·모양만 바뀐 것 (결정 사항과 부딪히지 않음)

  • 목록: 탭 "이슈별", 설명 "답한 변경은 7일 뒤에 목록에서 내려가요", 빈 화면 "지금 … 없어요 / 알려 드릴게요", 아래 줄 가운뎃점 빼기(feat(notify): 기준이 바뀌면 알릴 담당자를 고르는 함수 추가 (SIL-67) #7 이슈 제목 호윤 오후 3:40), 완료된 이슈는 "(완료된 이슈)" 글자 대신 [완료] 칩, 이전 값만 더 연한 회색, 되돌림 줄의 "내가 v2에 답한 이력이 있어요" 숨김
  • 상세: 값 상자 "이전 결정 사항 / 현재 결정 사항", 바뀐 이유를 이슈 카드 밖 제목 아래로, [되돌림]은 제목 옆에서 빼고 "현재 결정 사항" 상자 옆, [완료된 이슈] 칩 대신 이슈 카드에 [완료], 고르기 전 답 버튼 연한 파랑·파란 글자, "다음 변경 ›" 화살표 작게, 답한 뒤 알림 "답변을 완료했어요.", 지금 보는 이슈 칩에는 제목을 띄우지 않음, 지금 해야 할 일 설명 "내 이슈가 바뀐 결정 사항에 영향을 받는지 답해주세요"
  • 알림 꺼짐 안내: 그림 한 장(notify-guide.svg, 지움)을 잘라 쓰던 것을 시안의 그림 3장(public/notify-step-1~3.svg) + 그림 아래 진짜 글자로, "…클릭합니다/변경합니다" → "…클릭해요/변경해요"

왜 바꿨나요

신지님 시안이 10/15 오후에 바뀌었어요. 결정 사항과 다른 부분은 T1(직접 요청)으로 먼저 바꿔 등록했어요 (팀 DB, 연결된 이슈 없음).

  • app.detail.change-letter 새로 등록: 상세 위 칩은 결정 사항과 내 이슈 한 쌍마다 하나씩 두고 받은 순서대로 변경 A, 변경 B처럼 부른다
  • app.card.three-steps v2 → v3: 단계 이름 결정 사항 변경, 확인 완료, 이슈 반영 (v2가 이미 이슈 반영이었는데 코드는 작업 반영이었어요)
  • app.card.found-in-pr-code-only v1 → v2: 출처 칩 하나를 "PR에서 찾은 변경"으로, 따로 칩을 더 붙이지 않음
  • notify.pending.renotify-hint v1 → v2: 안내 문구 변경

어떻게 확인했나요

  • pnpm check 통과 (vitest 238 통과·1 건너뜀), check_wiki.py 오류 0개
  • 예시 데이터(5174)에서 대체됨 상세의 노란 안내, 출처 칩([AI 대화] [Slack] [PR에서 찾은 변경] [Figma]), 다시 반영 안내 문구 확인
  • 예시 데이터(5174)에서 목록(이슈별 탭, 설명, [완료] 칩, 연한 이전 값, 아래 줄)과 되돌림 상세(바뀐 이유 위치, 이전/현재 결정 사항, [되돌림] 위치, 이슈 카드 [완료], 연한 파랑 답 버튼) 확인

직접 확인하지 못한 부분

  • 상세 위 칩: 예시 데이터에서 변경 A~G와 마우스 올림 글자를 확인했어요. 칩이 많아지면 가로로 넘겨 봐야 하는 건 그대로예요. 7일이 지나 목록에서 빠지는 변경이 생기면 뒤 글자가 하나씩 당겨질 수 있어요.

  • 터미널 알림(packages/notify, 민수님)은 아직 예전 다시 반영 안내 문구예요. notify.pending.renotify-hint는 터미널에도 적용돼서 맞춰야 해요.

  • 답할 새 버전 카드가 없을 때의 상세 안내 문구(목록과 같은 문구로 둠)는 신지님 시안에 없는 경우라 신지님 확인 전이에요.

  • 알림 꺼짐 안내(권한이 '차단'일 때 펼치는 그림)는 화면에서 직접 열어 보지 않았어요. 제 브라우저는 알림이 허용돼 있어서 안내가 뜨지 않아요.

  • 답한 뒤 화면의 진행 단계와 "이슈에 반영했어요" 버튼은 예시 데이터로 직접 눌러 보지 않았어요.

🤖 Generated with Claude Code

space-angel and others added 6 commits October 15, 2026 23:17
app.card.three-steps 2→3, app.card.found-in-pr-code-only 1→2, notify.pending.renotify-hint 1→2

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
"…에서 바꿈"을 빼고 [AI 대화] [코드] [Slack] 등 칩 하나만 붙임. AI가 찾았고 출처가 코드면
[PR에서 찾은 변경]. 모두 같은 모양이고 AI 대화에만 반짝 아이콘 (app.card.found-in-pr-code-only v2)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- 진행 단계: 결정 사항 변경 → 확인 완료 → 이슈 반영, "이슈에 반영했어요" (app.card.three-steps v3)
- 다시 반영 안내: "이전 결정 사항으로 작업 중이었다면 최신 결정 사항을 반영해 주세요" (notify.pending.renotify-hint v2)
- 대체됨 상세: 회색 상자 대신 "지금 해야 할 일" 옆 노란 안내, 안내 상자는 노란 바탕·검정 글자
- 타입명세서·README 이름도 함께 고침

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
탭 "이슈별", 설명 "답한 변경은 7일 뒤에 목록에서 내려가요", 빈 화면 "지금 … 없어요 / 알려 드릴게요",
아래 줄 가운뎃점 빼기, 완료된 이슈는 [완료] 칩, 이전 값만 더 연하게, 되돌림 줄의 답한 이력 숨김

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
값 상자 "이전 결정 사항 / 현재 결정 사항", 바뀐 이유를 제목 아래로, [되돌림]은 현재 결정 사항 상자 옆,
[완료]는 이슈 카드, 고르기 전 답 버튼은 연한 파랑, 다음 변경 화살표 작게, 알림 "답변을 완료했어요.",
지금 보는 이슈 칩에는 제목을 띄우지 않음, 지금 해야 할 일 설명 "내 이슈가 바뀐 결정 사항에…"

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
신지님 시안(10/15)의 notify-step-1~3.svg를 쓰고, 그림 아래 단계 글자를 텍스트로 보여 줌 ("…해요"체).
그림 한 장을 잘라 쓰던 notify-guide.svg는 지움

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@space-angel space-angel changed the title fix(web): 신지님 시안(10/15)에 맞춰 진행 단계·출처 칩·안내 문구 고침 (SIL-74) fix(web): 나에게 온 변경 화면을 신지님 시안(10/15)에 맞춤 (SIL-74) Oct 15, 2026
space-angel and others added 3 commits October 15, 2026 23:26
새 버전 카드가 있으면 시안 문구 "최신 버전의 확인할 변경에 답해 주세요", 자동으로 닫혀 없으면
목록과 같은 "새 버전으로 대체됐다가, 내가 이미 확인한 값으로 돌아왔어요. 답하지 않아도 돼요"

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
app.detail.change-letter new→1 (T1 직접 요청, AI 대화)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
이슈로 묶어 "이슈 #7" 칩 하나에 다른 결정 사항이 섞이던 것을 결정 사항과 내 이슈 한 쌍마다 칩 하나로 나눔.
받은 순서대로 변경 A, 변경 B… 라서 답하거나 새 변경이 와도 이미 붙은 글자는 그대로예요.
마우스를 올리면 결정 사항 제목 · 이슈 · 상태 (app.detail.change-letter)

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

Copy link
Copy Markdown

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

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

@space-angel 이 PR에서 결정 사항이 여러 개 바뀐 것 같아요. 한 번에 하나만 저장해서 직접 확인을 부탁드려요.

app.card.three-steps 카드의 진행 단계는 결정 사항 변경, 확인 완료, 이슈 반영 세 개다

  • 지금 값: 카드의 진행 단계는 결정 사항 변경, 확인 완료, 이슈 반영 세 개만 보여 주고 열어봄과 처리 완료는 만들지 않는다
  • 바뀐 것 같은 값: 카드의 진행 단계는 결정 사항 변경, 확인 완료, 이슈 반영 세 개만 보여 주고 열어봄과 처리 완료는 만들지 않는다
  • 코드에서 바뀐 것: 상세 화면의 진행 단계 이름이 '새 변경, 확인 완료, 작업 반영'에서 '결정 사항 변경, 확인 완료, 이슈 반영'으로 바뀌었고, 결정 사항에는 이미 이 값이 적혀 있어요.
  • PR에 적힌 이유: 신지님 시안이 10/15 오후에 바뀌었어요.
-const STEPS = ['새 변경', '확인 완료', '작업 반영'];
+const STEPS = ['결정 사항 변경', '확인 완료', '이슈 반영'];
-        <Chip tone="ai">작업 반영</Chip>
+        <Chip tone="ai">이슈 반영</Chip>

app.card.found-in-pr-code-only "PR에서 찾은 변경" 출처 칩은 출처가 코드일 때만 보여 준다

  • 지금 값: 카드의 출처 칩은 AI가 찾았고 출처가 코드인 결정 사항에만 "PR에서 찾은 변경"으로 보여 주고 따로 칩을 더 붙이지 않는다
  • 바뀐 것 같은 값: 카드의 출처 칩은 AI가 찾았고 출처가 코드인 결정 사항에만 "PR에서 찾은 변경"으로 보여 주고 따로 칩을 더 붙이지 않는다
  • 코드에서 바뀐 것: 따로 붙던 'PR에서 찾음' 칩을 없애고, AI가 찾았고 출처가 코드면 출처 칩 하나를 'PR에서 찾은 변경'으로 보여 주게 바뀌었고, 결정 사항에는 이미 이 값이 적혀 있어요.
  • PR에 적힌 이유: 신지님 시안이 10/15 오후에 바뀌었어요.
-  return `${SOURCE_CHIP_LABELS[change.standard.source]}에서 바꿈`;
+  return isFoundInPr(change)
+    ? 'PR에서 찾은 변경'
-          PR에서 찾음

notify.pending.renotify-hint 관계있다로 답한 사람에게만 최신 결정 사항을 반영하라는 안내를 붙인다

  • 지금 값: 같은 결정 사항이 다시 바뀌면 지난번에 관계있다로 답한 사람에게만 "이전 결정 사항으로 작업 중이었다면 최신 결정 사항을 반영해 주세요" 안내를 붙인다
  • 바뀐 것 같은 값: 같은 결정 사항이 다시 바뀌면 지난번에 관계있다로 답한 사람에게만 "이전 결정 사항으로 작업 중이었다면 최신 결정 사항을 반영해 주세요" 안내를 붙인다
  • 코드에서 바뀐 것: 다시 반영 안내 문구가 '지난번 변경을 반영하셨다면 다시 고쳐야 해요'에서 '이전 결정 사항으로 작업 중이었다면 최신 결정 사항을 반영해 주세요'로 바뀌었고, 결정 사항에는 이미 이 값이 적혀 있어요.
  • PR에 적힌 이유: 신지님 시안이 10/15 오후에 바뀌었어요.
-const REWORK_NOTICE = '지난번 변경을 반영하셨다면 다시 고쳐야 해요';
+const REWORK_NOTICE =
+  '이전 결정 사항으로 작업 중이었다면 최신 결정 사항을 반영해 주세요';
-                지난번 변경을 반영하셨다면 다시 고쳐야 해요

app.detail.change-letter 상세 위 칩은 변경 A, 변경 B처럼 부른다

  • 지금 값: 변경 상세 위 칩은 결정 사항과 내 이슈 한 쌍마다 하나씩 두고 받은 순서대로 변경 A, 변경 B처럼 부른다
  • 바뀐 것 같은 값: 변경 상세 위 칩은 결정 사항과 내 이슈 한 쌍마다 하나씩 두고 받은 순서대로 변경 A, 변경 B처럼 부른다
  • 코드에서 바뀐 것: 상세 위 칩이 이슈마다 '이슈 feat(notify): 기준이 바뀌면 알릴 담당자를 고르는 함수 추가 (SIL-67) #7'로 묶이던 것에서 결정 사항과 이슈 한 쌍마다 받은 순서대로 '변경 A, 변경 B'로 바뀌었고, 결정 사항에는 이미 이 값이 적혀 있어요.
  • PR에 적힌 이유: 결정 사항 ID는 너무 길고 정해진 번호가 없어서
-  const groups = groupByIssue(props.all);
+  const groups = groupByChange(props.all);
+            const label = `변경 ${changeLetter(index)}`;
-                <Pill isActive={isActive}>이슈 {first.issue.key}</Pill>
+                <Pill isActive={isActive}>{label}</Pill>

결정 사항은 그대로 두었어요. 바뀐 게 맞으면 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