Skip to content

Feature/home layout - #9

Merged
i3months merged 10 commits into
feature/hompagefrom
feature/home-layout
May 14, 2026
Merged

Feature/home layout#9
i3months merged 10 commits into
feature/hompagefrom
feature/home-layout

Conversation

@Jaeho-Site

@Jaeho-Site Jaeho-Site commented May 12, 2026

Copy link
Copy Markdown
Contributor

🎯 작업 목적

홈페이지 구성을 위한 기본적인 글로벌 레이아웃(헤더 및 푸터) 뼈대를 세팅합니다.

✨ 주요 변경 사항

  • 헤더 & 푸터 정의 (Core)
    • 홈페이지용 임시 글로벌 네비게이션(SiteNav) 컴포넌트 및 인덱스 추가
    • 홈페이지용 임시 글로벌 푸터(SiteFooter) 컴포넌트 및 인덱스 추가
  • 홈페이지 레이아웃 세팅
    • App 내 홈페이지 라우트 경로 추가
    • 글로벌 레이아웃만 포함된 홈페이지 기본 컴포넌트 구조 정의
  • 기타 개선 사항
    • 에러 바운더리 타입 네이밍 리팩토링
    • 신규 스타일 토큰 추가

📝 리뷰어 참고 사항

  • 본 PR은 홈페이지 레이아웃 설정을 위한 브랜치로, 내부를 구성하는 상세 위젯(Hero, CTA 등)들은 다음 PR(feature/home-widgets)에서 순차적으로 추가될 예정입니다.
  • 임시로 작성된 SiteNav와 SiteFooter가 전역 레이아웃을 확인해 주시면 됩니다.

@vercel

vercel Bot commented May 12, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
stackup Ready Ready Preview, Comment May 13, 2026 6:35am

i3months
i3months reviewed May 13, 2026
View reviewed changes
Comment thread frontend/src/shared/lib/async-boundary/ErrorBoundary.tsx
@@ -1,23 +1,23 @@
import { Component, type ErrorInfo, type ReactNode } from 'react';

interface Props {

@i3months i3months May 13, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

[단순 질문] 디렉토리명이 여기서는 AsyncBoundary 네요.
다른 곳에서는 site-footer 처럼 kebab-case 인데, 여기서는 PascalCase로 설정한 이유가 궁금합니다.

@Jaeho-Site Jaeho-Site May 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

오 확인하지 못했는데, 감사합니다. 하나로 통일하는 방향으로 수정할게요.

Comment thread frontend/src/widgets/site-footer/ui/SiteFooter.tsx
Stack Up
</div>
<p className="mt-4 text-sage-200 max-w-sm leading-relaxed">
IT 직군 멀티모달 AI 면접 시뮬레이터. GitHub 레포와 이력서를 분석해

@i3months i3months May 13, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

이런 텍스트 하드코딩은 지금처럼 처리하는게 좋을까요?
아니면 따로 뺀 다음 거기에 몰아넣는게 좋을까요?

@Jaeho-Site Jaeho-Site May 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

어딘가에 주석을 달아두기도 했는데요, widgets의 단순 뷰 컴포넌트의 경우엔 세그먼트를 분리하기 보단 하나의 컴포넌트 파일에 두는것이 가독성도 좋고 관리도 편하다고 생각했습니다.
features처럼 비즈니스 로직이 들어가거나 복잡도가 증가하면 분리하는것이 좋다고 생각해요. 의견 나눠보면 좋을 것 같네요.

@i3months

i3months commented May 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

@Jaeho-Site
고생하셨습니다.

csh1668 added a commit to csh1668/stackup that referenced this pull request May 13, 2026
@csh1668
Merge pull request Team-StackUp#9 from Team-StackUp/feature/home-layout
c0534a6 
Feature/home layout
@i3months
i3months merged commit 3892b07 into feature/hompage May 14, 2026
2 checks passed
This was referenced Jun 30, 2026
Merged
Merged
i3months added a commit to i3months/stackup that referenced this pull request Jun 30, 2026
@i3months @claude
fix(ux): UX 감사 2차 배치 — 답변 유실 방지·검증·리포트 재배치·스켈레톤
5506f1e 
UX 감사 후속(보류했던 중간 규모 6건).

- [Team-StackUp#9 데이터유실] 라이브 텍스트 답변 낙관적 업데이트 롤백: 답변은 WS fire-and-forget
  (ack 없음)이라 tempId별 10s 타임아웃 도입 — 시간 내 서버 반영(pendingAnswers
  content 매칭 소진)이 없으면 stuck 낙관적 메시지 제거 + 입력 복원(restoreDraft) +
  실패 토스트. 낙관적 상태를 COMPLETED→CREATED('전송 중')로. 해피패스(서버 COMPLETED
  대체)·dedup 보존, 언마운트 시 타이머 정리.
- [Team-StackUp#17] 세션 생성: maxQuestions >= generalQuestionCount 교차검증 + 인라인 안내,
  라벨 명료화('한 주제를 얼마나 파고들지', '꼬리질문 포함 전체 질문 최대 개수').
- [Team-StackUp#18] 피드백 리포트: 강점/개선/키워드/학습방향을 점수 직후로 끌어올림,
  직무적합도·이해도·첫인상을 '추가 평가' 그룹으로 묶어 '종합 점수 미반영' 면책 1회.
- [Team-StackUp#16] 세션 생성 페이지 '워크스페이스로' 복귀 링크 추가.
- [Team-StackUp#22] 터치 타깃: Stepper ± 및 DeliveryModeToggle min-h-11(>=44px).
- [Team-StackUp#12] 공용 ListSkeleton 도입, 5개 목록의 '불러오는 중…' 한 줄 → 고정높이 골격(CLS↓).

build 통과, lint 신규 에러 없음. 프론트 전용.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@i3months i3months mentioned this pull request Jul 1, 2026
Merged
i3months added a commit to i3months/stackup that referenced this pull request Aug 28, 2026
@i3months @claude
fix(interview): 자기소개 답변 오롤백 방지 — 롤백을 연결 끊김에만 적용
d5a3dea 
Team-StackUp#136(Team-StackUp#9 라이브 답변 롤백)의 회귀를 회의적 리뷰(3/3)가 발견해 수정.

문제: 낙관적 답변 프루닝 트리거는 SESSION_MESSAGE(→메시지 리페치→pendingAnswers
content 매칭)뿐인데, 자기소개(모든 세션 첫 답변)는 placeholder/에코 없이
SelfIntroAnsweredEvent 만 발행되고 첫 SESSION_MESSAGE 는 Pro 모델 질문 풀 생성
이후에야 온다. 풀 생성이 10초를 넘기면 성공한 자기소개 답변에 ACK 타이머가 발화 →
오롤백 + 입력 복원 + 실패 토스트, 재제출은 SESSION_INVALID_STATE 로 드롭되어 오류 루프.

수정: 타임아웃 시 연결이 'open'이면 전송 실패가 아니라 '서버가 아직 에코 전'이므로
롤백하지 않는다(이후 SESSION_MESSAGE 가 프루닝). 연결이 끊긴(closed/connecting)
경우에만 진짜 실패로 보고 롤백. connectionRef 미러 추가. 일반 꼬리질문(즉시 에코)은
무영향, 진짜 전송 실패(연결 끊김) 안전망도 유지.

build 통과, lint 신규 에러 없음.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@i3months i3months i3months left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Jaeho-Site @i3months