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

[QA 중감점검] 공개상담 관련한 이슈 해결 #281

Merged
merged 30 commits into from
May 4, 2024

Conversation

rmdnps10
Copy link
Contributor

@rmdnps10 rmdnps10 commented May 3, 2024

Checklist


  • 올바른 브랜치에 PR을 보내도록 설정하였나요?
  • PR의 라벨을 올바르게 달았나요?

Description


QA 전에, 공개상담과 관련되어 있는 이슈들을 정리해서 해결했습니다.
자세한 이슈 내용들은, #280 에서 확인부탁드립니다!

Related Issues


관련된 이슈 number를 작성해주세요. #280

To Reveiwer


리뷰어가 주목해야할 사항들을 작성해주세요

kyuhho and others added 28 commits March 28, 2024 21:57
[Hotfix] 시간 선택 모달 디바이스에 따라 스타일 깨짐 오류
Revert "[Hotfix] 시간 선택 모달 디바이스에 따라 스타일 깨짐 오류"
…ix/time-select

Revert "Revert "[Hotfix] 시간 선택 모달 디바이스에 따라 스타일 깨짐 오류""
[0429 수정사항 반영] 메인 브랜치 배포
@rmdnps10 rmdnps10 added Bug 기능 오류 발생 및 픽스 Chore 자잘한 수정 labels May 3, 2024
@rmdnps10 rmdnps10 self-assigned this May 3, 2024
Copy link

netlify bot commented May 3, 2024

Deploy Preview for candid-semolina-d0db42 ready!

Name Link
🔨 Latest commit 91049a4
🔍 Latest deploy log https://app.netlify.com/sites/candid-semolina-d0db42/deploys/6635ad460e377b00084289f1
😎 Deploy Preview https://deploy-preview-281--candid-semolina-d0db42.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@rmdnps10 rmdnps10 added API API 관련 Stage-2/code-review-request work done and CI passed labels May 3, 2024
@@ -55,7 +55,7 @@ function MainQuestionSection() {
<SaveResizeIcon />
<Caption1 color={Grey2}>{card?.totalScrap}</Caption1>
</ButtonItem>
</ButtonList>
</ButtonList> */}
Copy link
Contributor

Choose a reason for hiding this comment

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

해당 주석이 추후에 사용되는 코드이면 주석으로 TODO: use later 이런식으로 표기해주고, 사용되지 않는다면 지워주시면 좋을거 같아요!

Comment on lines 106 to 123
const ButtonList = styled.div`
display: flex;
gap: 1.2rem;
`;
// const ButtonList = styled.div`
// display: flex;
// gap: 1.2rem;
// `;

const ButtonItem = styled.div`
border-radius: 0.8rem;
background: ${Grey6};
display: flex;
padding: 0.6rem 1.2rem 0.6rem 0.6rem;
align-items: center;
gap: 0.4rem;
`;
// const ButtonItem = styled.div`
// border-radius: 0.8rem;
// background: ${Grey6};
// display: flex;
// padding: 0.6rem 1.2rem 0.6rem 0.6rem;
// align-items: center;
// gap: 0.4rem;
// `;

const SaveResizeIcon = styled(SaveIcon)`
width: 2rem;
height: 2rem;
`;
// const SaveResizeIcon = styled(SaveIcon)`
// width: 2rem;
// height: 2rem;
// `;
Copy link
Contributor

Choose a reason for hiding this comment

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

마찬가지로 사용되지 않는 주석이면 지워주세요!

@rmdnps10 rmdnps10 merged commit f059fdd into dev May 4, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API API 관련 Bug 기능 오류 발생 및 픽스 Chore 자잘한 수정 Stage-2/code-review-request work done and CI passed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants