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

Revert "fix(view): AuthorBarChart에서 Author 10명 넘어가는 경우 나머지는 others로 처리" #554

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

HIITMEMARIO
Copy link
Contributor

@HIITMEMARIO HIITMEMARIO commented Jul 23, 2024

잘못 올라간 커밋 취소하기 위한 pr 입니다...! 승인 해주시면 수정 후 다시 pr 하도록 하겠습니다!
Reverts #548

@HIITMEMARIO HIITMEMARIO requested review from a team as code owners July 23, 2024 09:45
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.

복구하느라 고생하셨습니다!!

PR description에는 간단하게라도 작업 내용을 좀 더 적어주시면 좋겠습니다!

@HIITMEMARIO HIITMEMARIO self-assigned this Jul 23, 2024
);
console.log("reducedOtherAuthors", reducedOtherAuthors);
data = [...topAuthors, reducedOtherAuthors];
data = data.slice(0, 10);
Copy link
Contributor

Choose a reason for hiding this comment

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

고생많으셨습니다!!👍👍👍
추후에 변수명을 data보다 더 자세히 지어봐도 좋을 것 같네요!

@HIITMEMARIO HIITMEMARIO merged commit b38016a into main Jul 23, 2024
2 checks passed
@ytaek ytaek added this to the v0.7.0 milestone Aug 10, 2024
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.

3 participants