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

課題ある時のみカメラオンにするように修正 #70

Merged
merged 1 commit into from
Nov 16, 2024

Conversation

Kubosaka
Copy link
Collaborator

@Kubosaka Kubosaka commented Nov 16, 2024

User description

概要

変更内容

  • デフォルトをお題終了ページにして、お題ある時に表示する

動作確認

  • どのような手順で動作確認を行ったのか記述してください。

関連するIssue

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

備考

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

PR Type

enhancement


Description

  • カメラのアクティベーションロジックを変更し、未回答の課題がある場合のみカメラをオンにするように修正しました。
  • isActiveのデフォルト状態をfalseに変更しました。
  • AssignmentBadge.tsxでのインポート文をtypeキーワードを使用して明確化しました。

Changes walkthrough 📝

Relevant files
Enhancement
page.tsx
Update camera activation logic based on assignment status

app/src/app/camera/page.tsx

  • Changed default isActive state to false.
  • Modified logic to activate camera only when there are unanswered
    assignments.
  • Removed redundant condition checks for isActive state.
  • +7/-8     
    AssignmentBadge.tsx
    Update import statement for type clarity                                 

    app/src/components/AssignmentBadge.tsx

  • Changed import statement to use type keyword for todayAssignment.
  • +1/-1     

    💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

    Copy link

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

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

    Logic Change
    isActiveのデフォルト値がfalseに変更され、未回答の課題がある場合のみカメラがアクティブになるロジックが追加されました。この変更が意図した通りに機能するか、特に条件分岐の部分を確認する必要があります。

    Import Statement
    todayAssignmentのインポート文がtypeキーワードを使用して明確化されましたが、他の部分での影響を確認する必要があります。

    Copy link

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    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.

    良さげ
    スマホからは本番じゃないとわからんので、一旦マージで

    @Kubosaka Kubosaka merged commit 14a5858 into main Nov 16, 2024
    2 checks passed
    @nose221834 nose221834 deleted the feat/kubosaka/fix-camera-page branch November 16, 2024 06:25
    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