Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feat] 지원서 작성 페이지 레이아웃 구성 #1251 #1258

Merged

Conversation

PHJoon
Copy link
Contributor

@PHJoon PHJoon commented Mar 5, 2024

📌 개요

  • 지원서 작성 페이지 레이아웃입니다.
  • /recruit/application으로 직접 접근하시면 됩니다.

💻 작업사항

  • 아래 테스트 버튼 누르시면 mock 데이터 바꿔볼 수 있습니다.
  • 피그마 보고 일단 "지원서 작성, 지원서 수정" 이 부분을 sticky로 구현해놨는데, 맞게 구현한건지는 모르겠습니다.
  • 테스트 버튼은 나중에 지우겠습니다.
스크린샷 2024-03-05 오전 11 21 19 스크린샷 2024-03-05 오전 11 21 35

✅ 변경로직

yoouyeon and others added 28 commits February 26, 2024 20:11
@PHJoon PHJoon added user 일반 사용자가 사용하는 서비스에서 발생한 issue recruit 모집 기능 라벨 labels Mar 5, 2024
@PHJoon PHJoon self-assigned this Mar 5, 2024
@PHJoon PHJoon changed the base branch from main to recruit March 5, 2024 02:36
@PHJoon PHJoon linked an issue Mar 5, 2024 that may be closed by this pull request
2 tasks
@yoouyeon yoouyeon requested review from a team, yoouyeon and greatSweetMango and removed request for a team March 5, 2024 03:04
Copy link
Member

@yoouyeon yoouyeon left a comment

Choose a reason for hiding this comment

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

리뷰 늦어서 죄송합니다.. 🙇‍♀️ 하나만 확인해주시고 conflict 난거 해결 부탁드려요! 수고하셨습니다~!

Comment on lines 6 to 12
const { data, isLoading } = useQuery<IRecruitmentDetail>(
['recruitDetail', id],
async () => {
const res = await mockInstance.get(`/recruitments/${id}`);
return res.data;
}
);
Copy link
Member

Choose a reason for hiding this comment

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

지원서 데이터는 페이지 랜더링 시에 한번만 요청하면 될 것 같은데 useQuery에 따로 옵션을 주지 않으면 되게 자주 데이터를 새로 불러오게 되거든요 (저도 옵션을 설정해놨어야 했는데 지금 보니 빼먹었네요;;)

위 글들 읽어봤는데 refetchOnWindowFocus 옵션 정도는 false로 설정해도 되지 않을까 싶어요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정했습니다. 새로운 내용 배워갑니다! 👍

top: 0;
width: 25rem;
background-color: rgba(246, 246, 246, 0.9);
backdrop-filter: blur(5px);
Copy link
Member

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@parksangmin1543 parksangmin1543 left a comment

Choose a reason for hiding this comment

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

역시 부탁드리길 잘했네요. 감사합니다.

Copy link
Member

@yoouyeon yoouyeon left a comment

Choose a reason for hiding this comment

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

변경 사항 확인했습니다~! 수고하셨어용

@PHJoon PHJoon merged commit 7f03925 into recruit Mar 7, 2024
@PHJoon PHJoon deleted the Feat/1251-지원서-작성-페이지-레이아웃-구성 branch March 7, 2024 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
recruit 모집 기능 라벨 user 일반 사용자가 사용하는 서비스에서 발생한 issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 지원서 작성 페이지 레이아웃 구성
3 participants