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

fix: コミットデータが0の時の表示を追加 #23

Merged
merged 1 commit into from
Dec 29, 2024
Merged

Conversation

namidapoo
Copy link
Owner

issue

変更内容

確認したこと

スクリーンショット

補足事項

PR 時のセルフチェック

  • 関数や変数の命名は一目で分かるものになってますか?
  • コメントは適切な量で、誰が見ても分かるコメントになってますか?
  • PR 内容に関するテストは書きましたか?
  • Issue の完了の定義は満たせていますか?

@Copilot Copilot bot review requested due to automatic review settings December 29, 2024 11:43
Copy link

TODOコメントは見つかりませんでした。

@namidapoo namidapoo enabled auto-merge December 29, 2024 11:43
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

app/recap/components/graph/languages.tsx:101

  • Using template literals inside JSX can lead to unexpected results. Consider using JSX expressions instead, like this: {あなたが最も使用した言語は <b>${limitedData[0].language}</b> です。}
: `あなたが最も使用した言語は <b>${limitedData[0].language}</b> です。`

@@ -61,7 +61,7 @@ export const OverView: FC<Props> = async ({ data }) => {
</CardHeader>
<CardContent className="pr-24 space-y-1">
<div className="text-sm text-muted-foreground font-bold italic truncate">
"{data.userProfile.bio}"
{data.userProfile.bio && "{data.userProfile.bio}"}
Copy link
Preview

Copilot AI Dec 29, 2024

Choose a reason for hiding this comment

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

The string interpolation is incorrect. It should be {data.userProfile.bio} instead of "{data.userProfile.bio}".

Suggested change
{data.userProfile.bio && "{data.userProfile.bio}"}
{data.userProfile.bio && `${data.userProfile.bio}`}

Copilot is powered by AI, so mistakes are possible. Review output carefully before use.

Positive Feedback
Negative Feedback

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Please select one or more of the options
Copy link

Deploying dev-recap with  Cloudflare Pages  Cloudflare Pages

Latest commit: b95750a
Status: ✅  Deploy successful!
Preview URL: https://b6d8e951.dev-recap.pages.dev
Branch Preview URL: https://fix-no-data.dev-recap.pages.dev

View logs

@namidapoo namidapoo disabled auto-merge December 29, 2024 11:44
@namidapoo namidapoo enabled auto-merge December 29, 2024 11:44
@namidapoo
Copy link
Owner Author

@github-actions approve me

@namidapoo namidapoo merged commit b7fe8fb into dev Dec 29, 2024
5 checks passed
@namidapoo namidapoo deleted the fix/no-data branch December 29, 2024 11:45
@namidapoo namidapoo mentioned this pull request Dec 29, 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.

1 participant