Repository navigation
클래스 API와 Android·Figma 화면 정합성 반영 #79
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Roy-wonji
merged 39 commits into
develop
from
feature/class-api-midvote-alignment-20261028
Nov 9, 2026
Merged
Changes from all commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
f2c81df
클래스 중간 투표 API와 분기 재생 흐름 연결
Roy-wonji adfe235
클래스 생성 대상별 요청 키 분리
Roy-wonji d99adc0
클래스 API와 홈 디자인 정합성 반영
Roy-wonji 7cead4a
클래스 투표·청취·의견·결과 화면 Figma 정합성 반영
Roy-wonji f451f73
클래스 Figma 대조용 캡처 테스트와 추출 대상 추가
Roy-wonji 64707f0
클래스 화면 Figma 줄 높이·간격 2차 정합성 반영
Roy-wonji 09b1959
클래스 결과 내 활동 API 연동
Roy-wonji 7f4eec0
의견 중복 신고를 서버 오류 코드로 구분
Roy-wonji b6eabc0
클래스 리포트와 내가 남긴 의견을 내 활동 API로 연결
Roy-wonji a3c1716
클래스 의견 더보기 메뉴로 신고·수정·삭제 연결
Roy-wonji 96de226
클래스 의견 메뉴 점검 지적 반영
Roy-wonji e2a1880
클래스 의견 참여 판정을 내 의견 빈 응답에도 유지
Roy-wonji fe837df
운영자 대시보드 내 투표 결과 활동 행을 3열로 변경
Roy-wonji 8d23d20
내가 남긴 의견 화면을 닫으면 진행 중인 조회 취소
Roy-wonji cc95774
클래스 코디네이터 피드백 테스트의 uuid 의존성 오류 수정
Roy-wonji 13367d7
리포트 화면을 닫으면 투표 기록·인사이트 요청도 취소
Roy-wonji 16e1590
운영자 대시보드 내가 남긴 의견 문구를 리포트와 맞춤
Roy-wonji 316f478
내 의견 다시 시도 시 비어 있는 의견 목록도 다시 조회
Roy-wonji 1e8b7c4
이용권 상태값을 모르면 만료로 처리
Roy-wonji 3bb9296
공용 팝업에 안내 스타일과 이용권 안내 문구 추가
Roy-wonji ba48891
클래스 이용권 동작을 Android와 맞춤
Roy-wonji febb9fe
홈 Best 댓글 답글 아이콘 에셋 추가
Roy-wonji bec874b
분석 이벤트 section 에 best_comment 추가
Roy-wonji f2e0ada
홈 화면을 Android 개편 시안에 맞춤
Roy-wonji 6a8aaab
홈 Best 댓글 배지를 첫 줄 기준선에 겹쳐 행 높이를 Android와 맞춤
Roy-wonji 3caeb39
홈 실패 후 재진입 테스트가 effect 완료를 기다리도록 수정
Roy-wonji 5de25aa
홈 알림·시계·눈 아이콘 에셋 추가
Roy-wonji df3e8cf
DesignKit 에 Android 줄높이와 같은 줄 간격 글꼴 함수 추가
Roy-wonji d21cc0c
홈 재선택·새로고침·실패 안내 동작을 Android와 맞춤
Roy-wonji 1dcd03d
홈 상단 바·EDITOR PICK·배틀 카드·빈 화면을 Android 시안에 맞춤
Roy-wonji a24388f
홈 섹션 헤더·Best 배틀·Best 댓글을 Android 실제 줄높이로 다시 맞춤
Roy-wonji d8c939c
홈 스켈레톤을 Android HomeSkeleton 구성으로 교체
Roy-wonji 43070c4
홈 2차 정합 점검 지적 반영
Roy-wonji a0e4abc
홈 배틀 카드 부품을 공용 UI로 옮기고 닫기 아이콘 추가
Roy-wonji 9a45e71
추천 화면을 Android RecommendScreen 에 맞춤
Roy-wonji 4f52fbc
공용 탭·빈 상태·배지·상단 바 제목을 Android 컴포넌트에 맞춤
Roy-wonji 745bc5c
관점 목록 화면을 Android PerspectiveScreen 에 맞춤
Roy-wonji 5d50a72
공용 입력창·더보기 메뉴를 Android PickeCommentInput·PickeMoreMenu 에 맞춤
Roy-wonji e12c061
댓글 화면을 Android CommentScreen 에 맞춤
Roy-wonji File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -28,7 +28,11 @@ struct ClassCoordinatorFeedbackTests { | |
| removed.report = MockClassReport.figma | ||
| removed.sourceViewState = .ready | ||
| var state = ClassCoordinator.State() | ||
| state.routes.push(.detail(.init(room: room))) | ||
| state.routes.push(.detail(withDependencies { | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🟠 [P2] Major TestStore 외부에서 |
||
| $0.uuid = .incrementing | ||
| } operation: { | ||
| ClassDetailFeature.State(room: room) | ||
| })) | ||
| state.routes.push(.report(first)) | ||
| state.routes.push(.report(removed)) | ||
| let store = TestStore(initialState: state) { ClassCoordinator() } withDependencies: { | ||
|
|
@@ -69,7 +73,11 @@ struct ClassCoordinatorFeedbackTests { | |
| original.report = MockClassReport.figma | ||
| original.sourceViewState = .ready | ||
| var state = ClassCoordinator.State() | ||
| state.routes.push(.detail(.init(room: room))) | ||
| state.routes.push(.detail(withDependencies { | ||
| $0.uuid = .incrementing | ||
| } operation: { | ||
| ClassDetailFeature.State(room: room) | ||
| })) | ||
| state.routes.push(.report(original)) | ||
| let store = TestStore(initialState: state) { ClassCoordinator() } withDependencies: { | ||
| $0.classResultUseCase = generator | ||
|
|
@@ -116,7 +124,11 @@ struct ClassCoordinatorFeedbackTests { | |
| original.dashboard = MockClassOwnerDashboard.figma | ||
| original.sourceViewState = .ready | ||
| var state = ClassCoordinator.State() | ||
| state.routes.push(.detail(.init(room: room))) | ||
| state.routes.push(.detail(withDependencies { | ||
| $0.uuid = .incrementing | ||
| } operation: { | ||
| ClassDetailFeature.State(room: room) | ||
| })) | ||
| state.routes.push(.ownerDashboard(original)) | ||
| let store = TestStore(initialState: state) { ClassCoordinator() } withDependencies: { | ||
| $0.classResultUseCase = generator | ||
|
|
@@ -243,6 +255,13 @@ private actor SuspendedFeedbackGenerator: ClassResultInterface { | |
| throw ClassAIFeedbackError.generationFailed | ||
| } | ||
|
|
||
| func fetchMyActivity( | ||
| roomID _: Int, | ||
| cursor _: String? | ||
| ) async throws -> ClassMyActivity { | ||
| throw ClassAIFeedbackError.temporarilyUnavailable | ||
| } | ||
|
|
||
| private func suspend<T: Sendable>( | ||
| request: @escaping @Sendable (CheckedContinuation<T, Error>) -> Request | ||
| ) async throws -> T { | ||
|
|
||
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🟠 [P2] Major
경로: 마이 탭에서 이용권 화면에 진입 시 탭바 표시
원인: 이전 코드(
return false)는 이용권 화면이 루트에 있으면 항상 탭바를 숨겼습니다. 변경된 코드(return ticket.customAlert != nil)는 이용권 화면에customAlert이 있을 때만 탭바를 숨기므로, 일반적인 이용권 등록/개요 화면에서는 탭바가 계속 표시됩니다.수정 방향: 이용권 화면에서
customAlert이 없는 경우에도 탭바가 숨겨져야 하는지, 혹은 현재 의도대로customAlert이 있는 경우에만 숨기는 것이 맞는지 디자인/기획 의도를 재확인하여 탭바 표시 동작을 일관성 있게 맞춰야 합니다.