Skip to content

release: add navigable career progression - #54

Merged
codingnanyong merged 2 commits into
mainfrom
develop
Sep 11, 2026
Merged

release: add navigable career progression#54
codingnanyong merged 2 commits into
mainfrom
develop

Conversation

@codingnanyong

Copy link
Copy Markdown
Owner

요약

  • 직업별 월페이퍼의 원판과 랜드마크를 상세 지역 진입점으로 연결
  • 전직 후 직업별 가이드 NPC와 확대된 기술 지역 화면 추가
  • 1차 직업에서 ??? 2차 전직으로 이어지는 Career Path 계보 표시
  • 미래 전문 몬스터를 완성률에서 제외되는 도감 ??? 슬롯으로 추가
  • 저장 스키마 v2 및 v1 자동 마이그레이션 적용
  • README와 Career Path 설계 문서 갱신

검증

  • npm run test — 154 tests passed
  • npm run lint
  • npm run build
  • Feature PR #53의 PR policy, CI, Vercel, GitGuardian 통과

포함 PR

Claude 자동 리뷰 실패는 확인된 저장소 토큰/도구 권한 문제이며 코드 피드백은 생성되지 않았습니다.

@vercel

vercel Bot commented Sep 11, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
codigdex Ready Ready Preview Sep 11, 2026 7:23am UTC

@claude

claude Bot commented Sep 11, 2026

Copy link
Copy Markdown

Claude encountered an error after 1s —— View job


I'll analyze this and get back to you.

@codingnanyong
codingnanyong merged commit 36039c7 into main Sep 11, 2026
6 of 7 checks passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9cf583758e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 88 to +92
this.hud = this.createHud(backdrop);
if (this.activeChapter && this.activeMonster) this.createQuestActors();
if (this.activeChapter && this.activeMonster) {
this.createQuestActors();
} else if (this.selectedCareerId) {
this.createCareerAtlas(this.selectedCareerId);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep the career atlas below the HUD

When the selected career is frontend or data-engineer, the frontend-testing and monitoring landmark hit areas cover the top-right HUD coordinates, including the center of the Codigdex button at (890, 26). Because the atlas is created after the HUD, these interactive areas receive a normal button-center click and open career-region instead of the dex; give the HUD a higher input/render depth or constrain the overlapping landmark hit areas.

Useful? React with 👍 / 👎.

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