Skip to content

MyPayment requests: improve request expiration logic#67

Merged
armanddidierjean merged 4 commits into
devfrom
better-request-expiration
Apr 20, 2026
Merged

MyPayment requests: improve request expiration logic#67
armanddidierjean merged 4 commits into
devfrom
better-request-expiration

Conversation

@Rotheem

@Rotheem Rotheem commented Apr 19, 2026

Copy link
Copy Markdown

Description

Summary

Compute request expiration using creation date instead of relying on a type column

Additional Notes

Classification

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 🔨 Refactor (non-breaking change that neither fixes a bug nor adds a feature)
  • 🔧 Infra CI/CD (changes to configs of workflows)
  • 💥 BREAKING CHANGE (fix or feature that require a new minimal version of the front-end)
  • 😶‍🌫️ No impact for the end-users

Impact & Scope

  • Core functionality changes
  • Single module changes
  • Multiple modules changes
  • Database migrations required
  • Other: ...

Testing

  • 1. Tested this locally
  • 2. Added/modified tests that pass the CI (or tested in a downstream fork)
  • 3. Tested in a deployed pre-prod
  • 0. Untestable (exceptionally), will be tested in prod directly

Documentation

  • Updated the docs accordingly :
  • " Docstrings
  • # Inline comments
  • No documentation needed

@armanddidierjean armanddidierjean changed the title Improve request expiration logic MyPayment requests: improve request expiration logic Apr 19, 2026
@codecov

codecov Bot commented Apr 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
app/core/mypayment/utils_mypayment.py 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@armanddidierjean armanddidierjean force-pushed the better-request-expiration branch from 4874085 to b24268a Compare April 20, 2026 15:25
@armanddidierjean armanddidierjean merged commit be46bba into dev Apr 20, 2026
13 checks passed
@armanddidierjean armanddidierjean deleted the better-request-expiration branch April 20, 2026 16:00
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