Skip to content

Conversation

pcwadarong
Copy link
Member

@pcwadarong pcwadarong commented Aug 1, 2024

Description

  • Chart.js 를 사용하여 radar chart를 구현했습니다.
  • vite를 사용하여 세팅하였습니다.
  • npm create vite@latest
  • 간단한 read.me를 작성하였습니다.

Comments

  • 이젠 githru.에 컨트리뷰션 할 타이밍이긴 하지만 한번 PR 연습해보고 진행해보고 싶어서 뒤늦게나마 시도해봅니다 ㅎㅎ

@pcwadarong pcwadarong requested a review from ytaek as a code owner August 1, 2024 15:07
ytaek
ytaek previously approved these changes Aug 1, 2024
Copy link
Contributor

@ytaek ytaek left a comment

Choose a reason for hiding this comment

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

환영합니다!!!!!

차트 하나 살짝 만들어보고 시작하시면 더 잘하실 수 있을 꺼에요!!

@pcwadarong
Copy link
Member Author

캡처 간단하게 구현해보았습니다.

Copy link
Contributor

@BeA-Pro BeA-Pro left a comment

Choose a reason for hiding this comment

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

첫 PR이신가요? 축하드립니다! 피카츄가 쥐였다니... 충격이네요😅

Copy link
Contributor

@shgusgh12 shgusgh12 left a comment

Choose a reason for hiding this comment

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

반갑습니다 채원님! 앞으로 잘 부탁드립니다~~

Copy link
Contributor

@ytaek ytaek left a comment

Choose a reason for hiding this comment

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

LGTM입니다!!
다음에는 추가 작업하실 때는 기존 PR merge 하신 뒤에 하시면 좋을 것 같아요~

  • 기존에 받은 approve 가 날아가기도 하고, 짧게 끊어서 올리면 리뷰어들이 편할 수 있습니다 😄
  • 아니면 PR draft 라는 기능을 활용하면 되긴 합니다만, 일단 짧게 올리시는 게 좋습니다 : )

Comment on lines +13 to +15
ui, ux, interaction 쪽에 강한 프론트엔드 개발자가 되는게 꿈입니다! ㅎㅎ

함께 으쌰으쌰하면서 친목도 쌓고, 더 좋은 오픈소스 만들어 나갔으면 좋겠습니다. 읽어주셔서 감사합니다! No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

응원합니다!! 🚀🚀🚀🚀🚀

Comment on lines +12 to +19
ChartJS.register(
RadialLinearScale,
PointElement,
LineElement,
Filler,
Tooltip,
Legend,
);
Copy link
Contributor

Choose a reason for hiding this comment

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

다음에 기회가 되면 d3로 바로 도전해보시는 것도 좋겠습니다!!
어차피 chart들도 기본 rect, circle, line, text 등의 컴포넌들로 이뤄진 집합이라,
개인적으로 CSS보다 훨씬 쉬운 것 같습니다 😈😈😈

Copy link
Member Author

@pcwadarong pcwadarong Aug 3, 2024

Choose a reason for hiding this comment

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

오 그렇군요! 저는 그냥 제일 간단하다고 해서 가장 빨리 구현할 수 있을 것 같아 골랐는데 ㅎㅎ 다음에 한 번 도전해보겠습니다!!
PR 규칙도 새롭게 알았습니다 알려주셔서 감사합니다!
commit들 모은 담에 올려서 pr하고 또 다음에 모아서 pr하면 따로따로 올라가는 줄 알았네요..ㅎㅎ

Copy link
Contributor

Choose a reason for hiding this comment

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

ㅎㅎ 넵 머지하고 나면 따로 올라가게 됩니다!
머지 안하면 계속 해당 PR 뒤로 붙게 되구요~

Copy link
Member Author

Choose a reason for hiding this comment

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

넵! 숙지하겠습니다 🫡 githru vs ext에선 그렇게 진행해보겠습니다! :D

@pcwadarong pcwadarong merged commit 3b2385c into githru:main Aug 3, 2024
@pcwadarong
Copy link
Member Author

@BeA-Pro 넵 처음이라 많이 떨리네요 ㅎㅎ 감사합니다!! 햄스터같이 생기지 않았나요 🐹

@pcwadarong
Copy link
Member Author

@shgusgh12 저도 잘 부탁드립니다 화이팅화이팅~~!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants