Skip to content

Conversation

@hrideshmg
Copy link
Member

@hrideshmg hrideshmg commented Oct 14, 2025

Adds a new table to track when status update streak tracking should be paused for a batch.

Integrated the above with the miss-streak calculation and added a new attribute on_break to the on_date status update query.

@hrideshmg hrideshmg marked this pull request as ready for review October 14, 2025 07:24
Copilot AI review requested due to automatic review settings October 14, 2025 07:24
Copy link
Contributor

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.

Pull Request Overview

This PR implements status update breaks functionality, allowing users to define periods where status update streaks should not be affected by missed updates. The implementation includes database schema changes, model definitions, and query modifications to exclude break periods from streak calculations.

  • Adds database table and model structures for status breaks
  • Modifies consecutive miss calculation to exclude dates during break periods
  • Provides GraphQL mutation for creating status breaks

Reviewed Changes

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

File Description
src/models/status_update.rs Defines StatusBreakRecord and CreateStatusBreakInput structs
src/graphql/queries/member_queries.rs Updates consecutive miss query to filter out break periods
src/graphql/mutations/status_mutations.rs Adds create_status_break mutation
migrations/20251013082516_implement_status_breaks.sql Creates statusbreaks table

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@hrideshmg hrideshmg force-pushed the status_breaks branch 2 times, most recently from bc9b13a to eeeb97e Compare October 14, 2025 07:25
@hrideshmg hrideshmg requested a review from Copilot October 14, 2025 07:26
Copy link
Contributor

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.

Pull Request Overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Member

@ivinjabraham ivinjabraham left a comment

Choose a reason for hiding this comment

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

LGTM

@ivinjabraham ivinjabraham merged commit 124d83f into amfoss:develop Oct 25, 2025
2 checks passed
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.

2 participants