Feature/home layout#9
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
i3months
reviewed
May 13, 2026
| @@ -1,23 +1,23 @@ | |||
| import { Component, type ErrorInfo, type ReactNode } from 'react'; | |||
|
|
|||
| interface Props { | |||
Member
There was a problem hiding this comment.
[단순 질문] 디렉토리명이 여기서는 AsyncBoundary 네요.
다른 곳에서는 site-footer 처럼 kebab-case 인데, 여기서는 PascalCase로 설정한 이유가 궁금합니다.
Contributor
Author
There was a problem hiding this comment.
오 확인하지 못했는데, 감사합니다. 하나로 통일하는 방향으로 수정할게요.
| Stack Up | ||
| </div> | ||
| <p className="mt-4 text-sage-200 max-w-sm leading-relaxed"> | ||
| IT 직군 멀티모달 AI 면접 시뮬레이터. GitHub 레포와 이력서를 분석해 |
Member
There was a problem hiding this comment.
이런 텍스트 하드코딩은 지금처럼 처리하는게 좋을까요?
아니면 따로 뺀 다음 거기에 몰아넣는게 좋을까요?
Contributor
Author
There was a problem hiding this comment.
어딘가에 주석을 달아두기도 했는데요, widgets의 단순 뷰 컴포넌트의 경우엔 세그먼트를 분리하기 보단 하나의 컴포넌트 파일에 두는것이 가독성도 좋고 관리도 편하다고 생각했습니다.
features처럼 비즈니스 로직이 들어가거나 복잡도가 증가하면 분리하는것이 좋다고 생각해요. 의견 나눠보면 좋을 것 같네요.
Member
|
@Jaeho-Site |
csh1668
added a commit
to csh1668/stackup
that referenced
this pull request
May 13, 2026
Feature/home layout
This was referenced Jun 30, 2026
i3months
added a commit
to i3months/stackup
that referenced
this pull request
Jun 30, 2026
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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎯 작업 목적
홈페이지 구성을 위한 기본적인 글로벌 레이아웃(헤더 및 푸터) 뼈대를 세팅합니다.
✨ 주요 변경 사항
SiteNav) 컴포넌트 및 인덱스 추가SiteFooter) 컴포넌트 및 인덱스 추가App내 홈페이지 라우트 경로 추가📝 리뷰어 참고 사항
feature/home-widgets)에서 순차적으로 추가될 예정입니다.SiteNav와SiteFooter가 전역 레이아웃을 확인해 주시면 됩니다.