Skip to content

fix: increase order history to 180 days#537

Merged
bseverino merged 1 commit into
mainfrom
bseverino/RV2U-248-order-history
Apr 22, 2026
Merged

fix: increase order history to 180 days#537
bseverino merged 1 commit into
mainfrom
bseverino/RV2U-248-order-history

Conversation

@bseverino

Copy link
Copy Markdown
Member

Internal ticket: https://redventures.atlassian.net/browse/RV2U-248

Merge checklist:
Check off if complete or not applicable:

  • Documentation updated (not only docstrings)
  • Fixup commits are squashed away
  • Unit tests added/updated
  • Manual testing instructions provided
  • Noted any: Concerns, dependencies, migration issues, deadlines, tickets

Post-merge:

Copilot AI review requested due to automatic review settings April 21, 2026 17:13
@bseverino bseverino requested a review from a team as a code owner April 21, 2026 17:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the core configuration that controls how far back a user’s order history is fetched, extending the default cutoff window to support showing older purchases in the order history flow.

Changes:

  • Increase ORDER_HISTORY_PER_SYSTEM_REQ_CUTOFF_IN_DAYS from 90 to 180.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions

Copy link
Copy Markdown

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  commerce_coordinator/apps/core
  constants.py
Project Total  

This report was generated by python-coverage-comment-action

@bseverino bseverino merged commit 24663af into main Apr 22, 2026
12 checks passed
@bseverino bseverino deleted the bseverino/RV2U-248-order-history branch April 22, 2026 15:40
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.

3 participants