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

[migration]productdetail tsx 설정 #157

Merged
merged 2 commits into from
Apr 16, 2023
Merged

[migration]productdetail tsx 설정 #157

merged 2 commits into from
Apr 16, 2023

Conversation

bsw98
Copy link
Collaborator

@bsw98 bsw98 commented Apr 14, 2023

✨ 구현 기능

productdetail 부분 ts 마이그레이션 해놧는데
WriteInput이랑 Product 연결부분은 컴포넌트 만져야할 것 같아서
못했습니다 :d

✅ 작업 내용 상세

🙏 비고

const firestore = getFirestore();
const userCollection = collection(firestore, 'UserWrite');
const userRef = doc(userCollection, paramsId.id);
const userSnap = getDoc(userRef);
Copy link
Contributor

Choose a reason for hiding this comment

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

firebase 사용을 위해 바꾼 코드인거죠?

Comment on lines +48 to +50
useLayoutEffect(()=>{
if (scrollRef.current) {
scrollRef.current.scrollTop = scrollRef.current.scrollHeight;
Copy link
Contributor

Choose a reason for hiding this comment

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

스크롤 이슈 useLayoutEffect로 해결하셨군여! 굿굿

@DoctorK4 DoctorK4 added the 🧵migration Javascript to TypeScript label Apr 16, 2023
@DoctorK4 DoctorK4 merged commit 394122d into tsMigration Apr 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧵migration Javascript to TypeScript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants