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: 커피챗 상세페이지 Dialog 구현 #1619

Merged
merged 10 commits into from
Oct 31, 2024

Conversation

hayounSong
Copy link
Contributor

🤫 쉿, 나한테만 말해줘요. 이슈넘버

🧐 어떤 것을 변경했어요~?

커피챗 용 Dialog 뷰 및 기능, API 구현까지 완료했습니다

🤔 그렇다면, 어떻게 구현했어요~?

기존에 쓰던 Dialog에, 저번 고도화떄 만들어놓은 커스텀 Confirm을 사용해서 토스 라이브러리와의 충돌을 방지했습니당.

❤️‍🔥 당신이 생각하는 PR포인트, 내겐 매력포인트.

웹파트 행사를 위해 퀵 앤 러시로 2일만에 만들게 되어서 후에 조금은 리팩토링 하면 좋을 것 같습니다 ㅎ...

📸 스크린샷, 없으면 이것 참,, 섭섭한데요?

스크린샷 2024-10-28 오후 11 12 52

@hayounSong hayounSong self-assigned this Oct 30, 2024
Copy link

height bot commented Oct 30, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link

github-actions bot commented Oct 30, 2024

✨✨ 스토리북으로 확인하기 ✨✨

Copy link

github-actions bot commented Oct 30, 2024

🚀 프리뷰 배포 확인하기 🚀

https://fd0c7456.sopt-internal-dev.pages.dev

Copy link
Member

@pepperdad pepperdad left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!!@@
지금 prettier, stylelint 등 포맷터가 자동으로 적용이 안되는 느낌인데,,, 확인 부탁드릴게요!

<Text mt={30} typography='SUIT_24_B'>
커피챗 제안하기
</Text>
<Text style={{"textAlign":"center","lineHeight":"22px"}} mt={14} typography='SUIT_14_M' color={colors.gray300}>
Copy link
Member

Choose a reason for hiding this comment

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

p1:

Suggested change
<Text style={{"textAlign":"center","lineHeight":"22px"}} mt={14} typography='SUIT_14_M' color={colors.gray300}>
<Text style={{ textAlign: 'center', lineHeight: '22px' }} mt={14} typography='SUIT_14_M' color={colors.gray300}>

여기 인라인스타일로 된 부분 모두 따옴표 없어야할 것 같은데!!! 혹시 저장할 때 자동으로 안바뀌나요??

</Text>
<TextWrapper>
<StyledText mt={48} color={colors.white} >
회신 받을 본인 연락처 <span style={{"color":"#F77234"}}>*</span>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
회신 받을 본인 연락처 <span style={{"color":"#F77234"}}>*</span>
회신 받을 본인 연락처 <span style={{color: "#F77234"}}>*</span>

/>
<TextWrapper>
<StyledText mt={46} color={colors.white} >
무엇이 궁금하신가요? <span style={{"color":"#F77234"}}>*</span>
Copy link
Member

Choose a reason for hiding this comment

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

p1:

Suggested change
무엇이 궁금하신가요? <span style={{"color":"#F77234"}}>*</span>
무엇이 궁금하신가요? <span style={{color: "#F77234"}}>*</span>

Comment on lines 24 to 26



Copy link
Member

Choose a reason for hiding this comment

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

Suggested change

Copy link
Contributor Author

Choose a reason for hiding this comment

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

앗 제 로컬 eslint가 로직이 꼬여버린 것 같습니다. 수정해서 적용완료했어요!

@hayounSong hayounSong changed the base branch from main to feat/coffechat-detail October 31, 2024 12:10
@hayounSong hayounSong merged commit 77fc20a into feat/coffechat-detail Oct 31, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 커피챗 상세페이지 Dialog 구현
2 participants