Skip to content

[Fix/#152]: 우물 뒤로가기 버튼 수정, 첫메모 캐싱 오류 수정#200

Open
JIMIN1020 wants to merge 3 commits intodevfrom
feat/#152
Open

[Fix/#152]: 우물 뒤로가기 버튼 수정, 첫메모 캐싱 오류 수정#200
JIMIN1020 wants to merge 3 commits intodevfrom
feat/#152

Conversation

@JIMIN1020
Copy link
Contributor

📍 작업 내용

  • 우물 뒤로가기 버튼 수정

    • 타인의 우물인 경우 우물 리스트 아이콘 대신 뒤로가기 버튼이 렌더링 되도록 수정하였습니다.
  • 첫메모 캐싱 오류 수정

    • 첫메모 작성 후 메모 리스트로 돌아갔을 때 버튼 텍스트가 변화하지 않는 캐싱 오류를 해결하였습니다.
    • dynamic, revalidate를 설정하고, 추가로 headers()를 통해서 컴포넌트를 요청에 의존하는 컴포넌트로 만들었습니다. (on-demand로 매 요청마다 SSR 수행)

@JIMIN1020 JIMIN1020 requested a review from SebellKo August 12, 2025 13:00
@JIMIN1020 JIMIN1020 self-assigned this Aug 12, 2025
@JIMIN1020 JIMIN1020 added the feat 기능 구현 label Aug 12, 2025
@vercel
Copy link

vercel bot commented Aug 12, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Project Deployment Preview Comments Updated (UTC)
frolog Ready Preview Comment Aug 13, 2025 10:36am

Copy link
Contributor

@SebellKo SebellKo left a comment

Choose a reason for hiding this comment

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

지민님 코멘트 답변좀 부탁드립니다
지민님이 적용하신 개념들 잘 몰라서 찾아보니까 그렇다고 하는데 지민님 생각도 부탁드립니다 ~
일단 어프로브 해 둘게요 ~

Comment on lines +23 to +24
export const dynamic = 'force-dynamic';
export const revalidate = 0;
Copy link
Contributor

Choose a reason for hiding this comment

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

이거 두개 같이써야돼요 ?
force-dynamic만 있어도 되는거 아니에요 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fetch 같은거 안써서 force-dynamic만 있어도 돼요
혹시 몰라서 넣어놨습니다

export const revalidate = 0;

async function MemoPage({ params: { wellId, userId, bookId } }: Props) {
headers();
Copy link
Contributor

Choose a reason for hiding this comment

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

headers() 요것도 리턴값 사용하는거 없는데 필요한가요 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

사실 dynamic 옵션이 안먹혀서 이 방식으로 해결한거였어요. dynamic은 지워도 될듯
headers 를 호출하는 것만으로도 이 컴포넌트가 요청 헤더에 의존하는 것이 되기 때문에 동적으로 렌더링되는 원리입니다

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants