Skip to content

validate purchases on full cdr payment#1022

Draft
Luhmos wants to merge 1 commit into
mainfrom
total-pay-validate
Draft

validate purchases on full cdr payment#1022
Luhmos wants to merge 1 commit into
mainfrom
total-pay-validate

Conversation

@Luhmos

@Luhmos Luhmos commented Jun 25, 2026

Copy link
Copy Markdown
Member

Description

Currently, all purchases have to be validated in person. However, if paid on site, the same person collecting the payment has to then manually validate purchases, despite having checked them already before asking for the payment. Thus, we want, for ease of use, to validate all purchases automatically if the full payment has been made on site.

Summary

Added a check when an admin adds a payment to see if the full amount has been paid. If so, validate all purchases.

Issues/PR dependencies

Issues to be resolved

Required PRs

Changes Made

  • ...
  • ...

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

@Luhmos Luhmos self-assigned this Jun 25, 2026
@Luhmos
Luhmos marked this pull request as draft June 25, 2026 09:18
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