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

[장문원] week11 #391

Conversation

jangmoonwon
Copy link
Collaborator

요구사항

기본

  • 케밥 버튼을 누르면 팝오버가 보이게 해주세요.
  • [] 링크를 입력하고 “추가하기” 버튼을 누르면 “폴더에 추가” 모달이 뜨게 해주세요.
  • [] 폴더에 있는 버튼 “폴더 추가”, “공유”, “이름 변경”, “삭제”
팝오버에 있는 버튼 “삭제하기”, “폴더에 추가”
클릭시 해당하는 모달이 뜨게 해주세요.

심화

주요 변경사항

  • 지난번 코드리뷰 남겨주신 부분(푸터의 이미지 map함수 사용, getElapsedTime utill 생성) 리팩토링 해보았습니다.

스크린샷

멘토에게

  • 너무 늦게 드려 죄송합니다 ㅠㅠㅠ 리팩토링이랑 미션이 점점 버거워지네요ㅜㅜㅜ 미완성이지만 만든 만큼 멘토님께 리뷰 받고 싶습니다..ㅎㅎ

@jangmoonwon jangmoonwon requested a review from devym-37 May 6, 2024 13:35
@jangmoonwon jangmoonwon added 미완성🫠 죄송합니다.. 순한맛🐑 마음이 많이 여립니다.. labels May 6, 2024
@jangmoonwon jangmoonwon changed the title 장문원 [week11] [장문원] week11 May 6, 2024
Copy link
Collaborator

@devym-37 devym-37 left a comment

Choose a reason for hiding this comment

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

잘 작성해주셨습니다.👍
리팩토링과 과제 진행하는것이 어려운 부분이긴 합니다. 리팩토링은 작은부분을 고려해서 진행해보시고,
과제는 익숙해지는 연습으로 조금씩 해보시면 좋을 것 같습니다!
고생하셨습니다!

<img src={TwitterLogo} alt="twitter-logo" />
<img src={YoutubeLogo} alt="youtube-logo" />
<img src={InstagramLogo} alt="instagram-logo" />
{footerLogoList.map((item) => (
Copy link
Collaborator

Choose a reason for hiding this comment

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

footerLogoList를 외부에서 관리하시는 것은 좋은 것 같습니다

import youtubeIcon from "../../assets/youtube.svg";
import instagramIcon from "../../assets/instagram.svg";

const footerLogoList = [
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍


const cx = classNames.bind(styles);

export const Profile = ({ profile }) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

추후에는 해당 컴포넌트에서 필요한 데이터만 props로 받는것으로 수정해주시는 것도 좋을 것 같습니다. ex) email, profileImage 정보만 받는 것 입니다


export function useAsync(asyncFuction) {
const [data, setData] = useState(null);
function useAsync(asyncFuction) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

잘 작성해주셨습니다

}, []);

return { data, loading, error };
return [loading, error, data, wrappedFunction];
Copy link
Collaborator

Choose a reason for hiding this comment

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

return 타입을 {} 객체에서 [] 배열 상태로 바꾸신 이유가 있으실까요 ?

<button>삭제</button>
</div>
) : (
""
Copy link
Collaborator

Choose a reason for hiding this comment

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

"" 보다는 null로 해주시는 것이 좋을 것 같습니다

src={noImageLogo}
alt="카드 썸네일 없음"
/>
)}
Copy link
Collaborator

Choose a reason for hiding this comment

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

img 태그 하나만을 사용하고, 값 등에 삼항연산자를 사용하셔도 될 것 같습니다.
ex) src={item.image_source ? item.image_source : noImageLogo} 입니다

@@ -0,0 +1,38 @@
import { TIME } from "./constant";

export const getElapsedTime = (createdAt) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

util 폴더 하위에 파일로 분리해주신 부분 잘해주신 것 같습니다

@devym-37 devym-37 merged commit 7876122 into codeit-bootcamp-frontend:part2-장문원 May 6, 2024
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.

2 participants