Skip to content

feat(setting): 정적 FAQ 펼침·복원과 Nav3 이동 구현 (#785) - #1929

Closed
OnlyReadOneBook wants to merge 15 commits into
handoff/784-setting-1hyokfrom
handoff/785-setting-1hyok
Closed

feat(setting): 정적 FAQ 펼침·복원과 Nav3 이동 구현 (#785)#1929
OnlyReadOneBook wants to merge 15 commits into
handoff/784-setting-1hyokfrom
handoff/785-setting-1hyok

Conversation

@OnlyReadOneBook

@OnlyReadOneBook OnlyReadOneBook commented Sep 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

철회

정적 단일 FAQ·펼침·신규 entry는 제공 방식과 운영 콘텐츠 결정에 따라 재작성될 수 있어 이 구현 PR을 철회합니다. 해결 완료나 병합으로 처리하지 않습니다.

구현 이슈 #785도 not planned로 철회합니다. #1965에서 Afternote/Afternote-BE#246의 제공 방식과 실제 계약이 확정된 뒤 구현 범위를 다시 정합니다. 기존 UI 재사용을 선행 조건으로 삼지 않습니다.

고객센터 #1926도 운영 채널·연락처 승인 근거가 없어 전체 철회했습니다. #783에서 고객지원 제공 방식과 운영 정보를 결정합니다.

koongmai and others added 6 commits August 23, 2026 03:55
@koongmai
feat(setting): FAQ 화면 및 펼침 항목 구현
cec7bc3
@koongmai
Merge remote-tracking branch 'origin/develop' into feat/785
ee9b5ea
@koongmai @claude
fix(setting): FAQ 화면을 실제 내비게이션에 연결하고 문자열 이스케이프를 고친다
9f0e314 
- develop의 SettingNavGraph는 FaqScreen이 없어 onFaqClick = {} 로 죽은
  버튼 상태였다 (NoOpCallbackDefaultKonsistTest #1388 이 경계하는 패턴).
  SettingRoute.FaqRoute를 추가하고 실제 네비게이션으로 연결한다.
- faq_default_answer의 이스케이프되지 않은 작은따옴표가 aapt2 리소스
  컴파일을 깨뜨리고 있어 \' 로 이스케이프한다.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@koongmai @claude
test(setting): FaqScreen에 상호작용·스크린샷 테스트를 추가한다
870ec90 
- 질문 클릭 시 답변 토글, 뒤로가기 콜백 호출을 검증하는 Robolectric 테스트.
- 다른 setting 화면들과 동일하게 기본/컴팩트 스크린샷 테스트도 추가한다.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@koongmai @claude
test(setting): FaqScreen 스크린샷 레퍼런스 이미지를 추가한다
d25788a 
CI Screenshot 검증이 새 FaqScreenScreenshotTest의 레퍼런스 이미지가 없어
ScreenshotImageNotFoundException으로 실패했다. updateDebugScreenshotTest로
생성한 기본/컴팩트 baseline을 커밋한다.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@koongmai @claude
test(app): FAQ 네비게이션 배선을 실제 NavHost로 검증한다
661b7f0 
SettingNavGraph.kt·AppNavigationActions.kt가 navigation 패키지 파일이라
CI 정책상 androidTest.mode=selected가 필요하다. FAQ 메뉴 클릭 시
SettingRoute.FaqRoute로 이동하고 뒤로가기로 SettingHomeRoute에 복귀하는지
검증하는 계측 테스트를 추가한다.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@OnlyReadOneBook OnlyReadOneBook added the android-test PR head에서 Android Managed Device 계측 테스트 실행 label Sep 6, 2026
@OnlyReadOneBook OnlyReadOneBook self-assigned this Sep 6, 2026
@github-actions github-actions Bot added the sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다 label Sep 6, 2026
This was referenced Sep 6, 2026
Closed
Closed
Closed
@github-actions

github-actions Bot commented Sep 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

형제 충돌이 해소됐다 — 지금은 이 PR 과 같은 파일을 서로 다르게 고친 열린 PR 이 없다 (#1750).
다시 생기면 이 코멘트를 현재 상태로 고쳐 쓴다.

@github-actions github-actions Bot mentioned this pull request Sep 6, 2026
Closed
@OnlyReadOneBook OnlyReadOneBook mentioned this pull request Sep 6, 2026
Closed
@OnlyReadOneBook
OnlyReadOneBook changed the base branch from develop to handoff/784-setting-1hyok September 6, 2026 13:55
@github-actions github-actions Bot removed the sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다 label Sep 6, 2026
@OnlyReadOneBook
OnlyReadOneBook marked this pull request as ready for review September 6, 2026 15:04
@OnlyReadOneBook
OnlyReadOneBook marked this pull request as draft September 7, 2026 20:15
@OnlyReadOneBook OnlyReadOneBook mentioned this pull request Sep 7, 2026
Closed
5 tasks
@OnlyReadOneBook
OnlyReadOneBook marked this pull request as ready for review September 7, 2026 20:34
This was referenced Sep 7, 2026
Merged
Closed
@github-actions github-actions Bot added the sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다 label Sep 8, 2026
@github-actions github-actions Bot added the conflict base 와 충돌 — CI 가 트리거되지 않는다. base 를 병합하면 자동으로 떨어진다 label Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

이 PR 은 handoff/784-setting-1hyok 와 충돌 상태이고, 그래서 CI 가 돌지 않는다.

pull_request 워크플로는 base 와 head 를 합친 가상 머지 커밋(refs/pull/<N>/merge)에서
실행되는데, 충돌이면 GitHub 이 그 ref 를 만들지 못해 워크플로가 트리거조차 되지 않는다.
체크가 실패하는 것이 아니라 아예 생성되지 않으므로, 새 커밋을 올려도 no checks reported
인 채 최신 커밋이 한 번도 검증되지 않은 상태로 남는다 (#1028).

handoff/784-setting-1hyok 를 병합해 충돌을 풀면 검증이 다시 돈다. 해소되면 이 라벨은 자동으로 떨어진다.

@OnlyReadOneBook OnlyReadOneBook mentioned this pull request Sep 8, 2026
Closed
@github-actions github-actions Bot removed the conflict base 와 충돌 — CI 가 트리거되지 않는다. base 를 병합하면 자동으로 떨어진다 label Sep 8, 2026
@OnlyReadOneBook OnlyReadOneBook changed the title feat(setting): FAQ 화면 및 펼침 항목 구현 (#785) feat(setting): 정적 FAQ 펼침·복원과 Nav3 이동 구현 (#785) Sep 8, 2026
@github-actions github-actions Bot removed the sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다 label Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

❌ Screenshot test fail — baseline 과 CI rendered PNG 차이.

📦 Artifact: https://github.com/Afternote/Afternote-FE/actions/runs/34187191076 (페이지 하단 Artifacts → screenshot-test-results)

🔍 확인 위치 (artifact zip 내부):

  • diff PNG: preview/debug/diffs/...
  • rendered PNG: preview/debug/rendered/... (CI 환경에서 새로 그린 결과)
  • baseline PNG: repo <module>/src/screenshotTestDebug/reference/... (:feature:setting:presentation)

📝 조치 가이드:

  • 의도된 시각 변경 → 이 PR에 screenshot-baseline 라벨을 붙여 CI baseline 생성 요청
  • diff 가 anti-aliasing/font 차이로 보이면 docker 환경 변경 여부 (Dockerfile.screenshot) 점검

CI ↔ 로컬 환경 차이는 본 워크플로의 container 단계가 root fix 한다 (README 의 "Compose Preview Screenshot Testing (docker baseline)" 섹션 참조).

@OnlyReadOneBook OnlyReadOneBook added the screenshot-baseline CI 표준 환경에서 PR screenshot baseline 생성·적용 요청 label Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

📸 CI 표준 환경에서 screenshot baseline 4개를 생성·검증해 7a521bc 로 커밋했습니다.
실행 로그·생성 artifact: https://github.com/Afternote/Afternote-FE/actions/runs/34188020348
이미지 내용은 이 PR의 PNG diff에서 눈으로 최종 확인해 주세요. 필수 검사도 새 SHA에 다시 요청했습니다.

@github-actions github-actions Bot removed the screenshot-baseline CI 표준 환경에서 PR screenshot baseline 생성·적용 요청 label Sep 8, 2026
@OnlyReadOneBook
OnlyReadOneBook removed this pull request from stack #1966 September 12, 2026 22:19
This was referenced Sep 13, 2026
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Sadturtleman Sadturtleman Awaiting requested review from Sadturtleman

Assignees

@OnlyReadOneBook OnlyReadOneBook

Labels

android-test PR head에서 Android Managed Device 계측 테스트 실행

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@OnlyReadOneBook @Sadturtleman @koongmai