Skip to content

Dev to Main Sync#2588

Merged
prakashchoudhary07 merged 2 commits intomainfrom
develop
Mar 18, 2026
Merged

Dev to Main Sync#2588
prakashchoudhary07 merged 2 commits intomainfrom
develop

Conversation

@vinit717
Copy link
Member

@vinit717 vinit717 commented Mar 18, 2026

Date: 18-03-2026

Developer Name: @vinit717


Issue Ticket Number

#2038

PR going to sync

#2584

Description

Fix the flow to tag idle 7 day + correctly

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

Screenshot 1 Screenshot 2026-03-18 at 11 22 44 AM Screenshot 2026-03-19 at 12 06 11 AM Screenshot 2026-03-19 at 12 09 21 AM

* feat: add computeIdleDaysExcludingOOO function and update user status handling

* fix: improve error handling and data retrieval in Discord user status updates

* fix: enhance user status update logic and improve edge case handling in computeIdleDaysExcludingOOO

* feat: implement oooPeriods management and enhance idle day calculations in user status

* fix: update test expectations for user status data retrieval to use include.keys

* feat: add comprehensive documentation for group idle 7d+ workflow and enhance user status management with new OOO period handling

* chore: add logger import

* feat: Implement `idleWindowStartedAt` and refine OOO period calculation for accurate group idle 7d+ tracking, alongside new documentation.

* feat: add `resolveLastOooUntil` utility to manage OOO state transitions and update user status handling accordingly

* feat: initialize `lastOooUntil` property in user status updates to enhance OOO state management

* feat: enhance user status update logic by incorporating `previousUntil` for improved OOO state transition handling

* refactor: rename `idleWindowStartedAt` to `idleFrom` for consistency in user status management and update related tests

* refactor: rename variables for clarity in user status updates and improve time handling in idle user calculations

* refactor: update task status update endpoint and improve test assertions for user idle state

* feat: add idleFrom property to user status data for improved tracking of idle state
@coderabbitai
Copy link

coderabbitai bot commented Mar 18, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 1e76d3fe-0023-4674-8201-60bb2c0d5226

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

CodeRabbit can use TruffleHog to scan for secrets in your code with verification capabilities.

Add a TruffleHog config file (e.g. trufflehog-config.yml, trufflehog.yml) to your project to customize detectors and scanning behavior. The tool runs only when a config file is present.

@vinit717 vinit717 changed the title dev to main Dev to Main Sync Mar 18, 2026
@vinit717 vinit717 self-assigned this Mar 18, 2026
@vinit717
Copy link
Member Author

@prakashchoudhary07, here is the issue for fixing the edge case
#2590

@prakashchoudhary07 prakashchoudhary07 merged commit 196d90a into main Mar 18, 2026
6 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