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

userページにrateを繋ぎ込み #82

Merged
merged 2 commits into from
Nov 16, 2024
Merged

Conversation

hikahana
Copy link
Collaborator

@hikahana hikahana commented Nov 16, 2024

概要

  • このプルリクエストの目的や背景を簡単に説明してください。

変更内容

  • どのような変更を行ったのか具体的に記述してください。
    rateのgetAPIを繋ぎ込みした

動作確認

  • どのような手順で動作確認を行ったのか記述してください。
    userページを開いてrateの取得がされていればおkです
    make studioで値を入れないとです。
    ratePoint: 654
    rateId: 4
    でこんな感じになります

image

関連するIssue

  • 関連するIssue番号を記載してください。例: #123

備考

  • その他、レビュワーに伝えたいことがあれば記述してください。

Copy link

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 4 🔵🔵🔵🔵⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Possible Bug
nextRateの取得条件が変更されているため、意図した動作を確認する必要があります。特に、user.ratePointの代わりにuser.rate.maxRangeを使用していた部分が影響を与える可能性があります。

Code Smell
useEffect内でのAPI呼び出しが複数あり、エラーハンドリングが不足しています。これにより、ユーザー体験が悪化する可能性があります。

Code Smell
rankStylesの取得方法が冗長です。rate.rate.nameが存在しない場合のデフォルトスタイルの処理を見直す必要があります。

Copy link

PR Code Suggestions ✨

No code suggestions found for the PR.

@nose221834 nose221834 self-requested a review November 16, 2024 18:52
Copy link
Collaborator

@nose221834 nose221834 left a comment

Choose a reason for hiding this comment

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

yosa

@nose221834 nose221834 merged commit 5243e29 into main Nov 16, 2024
1 check passed
@nose221834 nose221834 deleted the fix/hikahana/rate-api-connect branch November 16, 2024 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants