Skip to content

feat(archive): 자료 상세를 /resources/:id 주소로 열기 (뒤로가기 → 자료실) - #444

Merged
choijunhuk merged 3 commits into
mainfrom
fix/archive-detail-url
Sep 9, 2026
Merged

feat(archive): 자료 상세를 /resources/:id 주소로 열기 (뒤로가기 → 자료실)#444
choijunhuk merged 3 commits into
mainfrom
fix/archive-detail-url

Conversation

@choijunhuk

Copy link
Copy Markdown
Collaborator

공지는 이미 /notices/:id 로 이동하지만 자료실은 같은 /resources 주소에서 상태로만 상세를 열어, 뒤로가기가 자료실 이전 페이지(예: 공지사항)로 튀었습니다. 상세 파일을 라우트에서 파생하도록 바꿔 열면 /resources/:id 로 push, 뒤로가기는 /resources 목록, 없는 id 는 목록으로 redirect. 스모크 테스트에 열기→URL→뒤로가기 시나리오 추가.

🤖 Generated with Claude Code

choijunhuk and others added 3 commits September 9, 2026 13:02
…o the list

Notices already navigate to /notices/:id, but the archive opened its detail
view in component state under the same /resources URL, so pressing back
from a file jumped to whatever page preceded the archive. The detail file is
now derived from the route: opening pushes /resources/:id, back lands on
/resources, and an unknown id redirects to the list.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
The desktop table fires the open handler from both the row and the cell,
which pushed /resources/:id twice and made browser back land on the same
file.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…e state

Both handlers run in the same event tick, so useParams still holds the
previous id for the second call; window.location is already updated by
the first pushState.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@choijunhuk
choijunhuk merged commit 1b6e998 into main Sep 9, 2026
4 checks passed
@choijunhuk
choijunhuk deleted the fix/archive-detail-url branch September 9, 2026 04:08
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