-
Notifications
You must be signed in to change notification settings - Fork 83
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
fix(view): #553 Author(other bar) 클릭시 해당 사용자들 관련 커밋내역이 나오지는 않는 문제 수정 #559
Changes from all commits
b140824
3335ca3
c1369be
2279c24
8998fea
4caf8da
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -51,12 +51,16 @@ export const convertNumberFormat = (d: number | { valueOf(): number }): string = | |
return d3.format("~s")(d); | ||
}; | ||
|
||
export const sortDataByAuthor = (data: ClusterNode[], author: string): ClusterNode[] => { | ||
export const sortDataByAuthor = (data: ClusterNode[], names: string[]): ClusterNode[] => { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 변수명을 바꾸신 이유가 있는지 궁금합니당!! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 기존에는 각각의 이름을 받았다면 수정 된 이후에는 배열을 받기때문에 names로 변경하게 되었습니다!! |
||
return data.reduce((acc: ClusterNode[], cluster: ClusterNode) => { | ||
const checkedCluster = cluster.commitNodeList.filter((commitNode: CommitNode) => | ||
commitNode.commit.author.names.includes(author) | ||
names.some((name) => commitNode.commit.author.names.includes(name)) | ||
); | ||
if (!checkedCluster.length) return acc; | ||
return [...acc, { nodeTypeName: "CLUSTER" as const, commitNodeList: checkedCluster }]; | ||
|
||
if (checkedCluster.length > 0) { | ||
acc.push({ nodeTypeName: "CLUSTER", commitNodeList: checkedCluster }); | ||
} | ||
|
||
return acc; | ||
}, []); | ||
}; |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,6 +2,7 @@ | |
|
||
.file-icicle-summary { | ||
text { | ||
fill: $white; | ||
fill: var(--primary-color); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 👍 |
||
filter: invert(100) grayscale(100) contrast(100); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 이 부분 궁금합니다!!!! (css를 잘 모릅니다 🥲) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 아, #558에 있던 내용이군요; 참고하였습니다. |
||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍👍👍👍👍👍👍👍👍👍