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: 연구 결과 피드백 없을 때 마크업 #492

Merged

Conversation

hyesungoh
Copy link
Member

🤔 해결하려는 문제가 무엇인가요?

스크린샷 2024-03-20 오후 11 12 25

요거 만들어야돼용

🎉 변경 사항

  • MobileHeader에서 사이드 메뉴바를 감출 수 있는 prop을 뚫었어용

import { HEAD_2_BOLD } from '~/styles/typo';

const ResultBasePage: FC = () => {
const { data, isLoading } = useGetSurveyIdByUserStatus();
Copy link
Member Author

Choose a reason for hiding this comment

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

요 훅 내부적으로 유저 상태를 enabled 옵션으로 쓰더라구요 ~

Comment on lines +35 to +41
useEffect(
function replaceWhenHasFeedback() {
if (!data) return;
if (data.all_feedback_count > 0) router.replace('/result');
},
[data, router],
);
Copy link
Member Author

Choose a reason for hiding this comment

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

피드백이 한 개라도 있는 사용자라면 리플레이스 함요 ~

Copy link
Member

@sumi-0011 sumi-0011 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~

@hyesungoh hyesungoh merged commit 76ba457 into feature/survey-base/handle-created-survey Mar 23, 2024
1 check passed
@hyesungoh hyesungoh deleted the feature/result/base branch March 23, 2024 12:39
hyesungoh added a commit that referenced this pull request Mar 23, 2024
* feat: 설문이 있을 때 핸들링

* feat: 연구 결과 피드백 없을 때 마크업 (#492)

* chore: 결과 베이스 이미지

* feat: mobile header props 뚫기

* feat: 결과 베이스
hyesungoh added a commit that referenced this pull request Mar 23, 2024
* chore: deps

* feat: 설문 없을 때 배경 이미지 적용

* feat: 설문이 있을 때 핸들링 (#491)

* feat: 설문이 있을 때 핸들링

* feat: 연구 결과 피드백 없을 때 마크업 (#492)

* chore: 결과 베이스 이미지

* feat: mobile header props 뚫기

* feat: 결과 베이스
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants