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/#363] Intro 페이지 추가 #364

Merged
merged 3 commits into from
Aug 29, 2024
Merged

[Feat/#363] Intro 페이지 추가 #364

merged 3 commits into from
Aug 29, 2024

Conversation

pepperdad
Copy link
Member

@pepperdad pepperdad commented Aug 28, 2024

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 작업 내용

📢 To Reviewers

  • qa방에 있는 디자인 이미지 넣고, 라우팅 처리만 추가했습니다!

📸 스크린샷

🔗 참고 자료

@github-actions github-actions bot added the ✨ FEAT 기능 구현 label Aug 28, 2024
Copy link

PR 작성하느라 고생 많았어요!! 라벨 잘 지정되었는지 확인 한 번 해 주기 🫶

Copy link

Copy link
Contributor

@imddoy imddoy left a comment

Choose a reason for hiding this comment

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

LGTM
ShawnTheSheepShawnGIF

Copy link
Contributor

Choose a reason for hiding this comment

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

p5) 역시 우리 리드야~ 벌써 pr을 올리잖아~ 하고 들어왔다가 깜짝아!

Comment on lines 1 to +2
import * as S from "./Loading.styled";

const LoadingAnimation = lazy(() => import("@components/commons/loading/LoadingAnimation"));
import LoadingAnimation from "./LoadingAnimation";
Copy link
Contributor

Choose a reason for hiding this comment

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

p5) 이거 이렇게 바꾼 이유가 궁금합니다!!🫣🫢😦

Copy link
Contributor

Choose a reason for hiding this comment

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

p5) 저도 궁금합니다 ! 아마 처음엔 초기 로딩 성능을 최적화하려고 동적 import를 한 것 같은데, 오히려 LoadingAnimation 이 늦게 import 되는 단점 같은 게 발생했던건가요..?

Copy link
Member Author

Choose a reason for hiding this comment

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

맞아요! 예전에 Next.js를 했을 때 애니메이션을 저 방식으로 lazy import 해왔는데!
지금 리액트에서 저렇게 사용하려고 하니까,,, 렌더링 될 때 오류가 발생하더라구요! 그래서 일단 지워줬습니다!

Copy link
Contributor

@sinji2102 sinji2102 left a comment

Choose a reason for hiding this comment

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

왕빠루다.... LGTM입니당

Copy link
Contributor

@ocahs9 ocahs9 left a comment

Choose a reason for hiding this comment

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

깔끔하네용 고생하셨습니다 ~~

Comment on lines 1 to +2
import * as S from "./Loading.styled";

const LoadingAnimation = lazy(() => import("@components/commons/loading/LoadingAnimation"));
import LoadingAnimation from "./LoadingAnimation";
Copy link
Contributor

Choose a reason for hiding this comment

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

p5) 저도 궁금합니다 ! 아마 처음엔 초기 로딩 성능을 최적화하려고 동적 import를 한 것 같은데, 오히려 LoadingAnimation 이 늦게 import 되는 단점 같은 게 발생했던건가요..?

@pepperdad pepperdad merged commit 7dcfb54 into develop Aug 29, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ FEAT 기능 구현
Projects
Status: 🎉Done
Development

Successfully merging this pull request may close these issues.

[ Feat ] 비트 소개 페이지 추가
4 participants