Skip to content

Conversation

halsk
Copy link
Contributor

@halsk halsk commented May 4, 2025

Report Status Check Logic Separation for TeamAnalysisResultPage

This PR implements Issue #4 by extracting the report status checking logic from the TeamAnalysisResultPage component into a dedicated custom hook called useReportStatus.ts.

Changes Made

  • Created a new hooks directory in frontend/src
  • Implemented useReportStatus.ts hook to handle:
    • Periodic status checking functionality
    • Pending analyses counting
    • Report status API calls
    • Refresh handling
  • Updated TeamAnalysisResultPage to use the new hook
  • Implemented proper cleanup to prevent memory leaks

Testing

  • Tested different URL patterns and analysis types
  • Verified that status checking works correctly
  • Verified that cleanup happens properly when navigating away

Resolves #4

Link to Devin run: https://app.devin.ai/sessions/2dcb4bdb671a48259ba5394f9c8b3b4d
Requested by: Hal Seki ([email protected])

@halsk halsk merged commit 5878042 into hackdays-io:main May 4, 2025
1 check passed
@halsk halsk deleted the devin/1746375333-status-check-hook branch May 6, 2025 05:19
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