Skip to content

feat: Home 페이지의 컨텐츠를 12기 모집 페이지로 대체#65

Open
dkkim0122 wants to merge 2 commits intodevelopfrom
feature/change-home-page-to-recruit-page
Open

feat: Home 페이지의 컨텐츠를 12기 모집 페이지로 대체#65
dkkim0122 wants to merge 2 commits intodevelopfrom
feature/change-home-page-to-recruit-page

Conversation

@dkkim0122
Copy link
Collaborator

작업 내용

  • 모집 시즌이 아닐 때는 메인 컨텐츠를 보여주고, 모집 시즌일 때는 모집 컨텐츠를 보여주어야 합니다.
  • feature flag 등을 사용하여 체계적으로 관리하는 것이 좋겠지만 이번주 내로 12기 모집 페이지를 제작해야 하는 상황이라 빠르게 구현하기 위해 state를 사용합니다.

@dkkim0122 dkkim0122 added the enhancement New feature or request label Apr 8, 2025
@dkkim0122 dkkim0122 requested a review from Gn0lee April 8, 2025 14:32
@dkkim0122 dkkim0122 self-assigned this Apr 8, 2025
Copy link
Collaborator

@Gn0lee Gn0lee left a comment

Choose a reason for hiding this comment

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

👍

"use client";
import { useState } from "react";
import HomeContainer from "@/app/_components/Home";
import RecruitContainer from "./_components/Recruit";
Copy link
Collaborator

Choose a reason for hiding this comment

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

절대 경로를 사용할 수 있을까요?

<ProjectSection />
<CrewMembersSection />
</>
<>{homePageMode === "home" ? <HomeContainer /> : <RecruitContainer />}</>
Copy link
Collaborator

Choose a reason for hiding this comment

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

상태를 내부에서 바꾸는것이 아니라면 굳이 state를 사용하지 않아도 괜찮을 것 같아요

지금 생각나는 것은 vercel의 edge config도 좋습니다.

https://vercel.com/docs/edge-config/get-started

하지만 이것은 나중에 생각해야할 것 같아요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

그렇네요 상태 사용할 필요가 없었네요ㅎㅎ 수정해놓겠습니다.
edge config 딱 저희가 찾는 거 같네요

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants