Skip to content

FIX: [PIDM-1340] - Receipt Recovery#141

Merged
FedericoRuzzier merged 69 commits intomainfrom
PIDM-1340-receipt-recovery
Mar 24, 2026
Merged

FIX: [PIDM-1340] - Receipt Recovery#141
FedericoRuzzier merged 69 commits intomainfrom
PIDM-1340-receipt-recovery

Conversation

@alessio-acitelli
Copy link
Copy Markdown
Collaborator

@alessio-acitelli alessio-acitelli commented Mar 16, 2026

List of Changes

  • Aligned project dependencies using BOM-based version management to ensure consistent library resolution and compatibility.
  • Improved receipt retrieval flow by reusing the list of TableEntity objects already returned by the filtered query, avoiding unnecessary additional lookups.
  • Updated controller and service unit tests to cover the new behavior and the revised filtering flow.
  • Fixed integration tests

Motivation and Context

The ticket required a technical stabilization and a functional fix on receipts retrieval.

Dependency management was aligned through BOM usage to prevent version mismatches and improve build consistency.

The receipts retrieval logic was improved to avoid unnecessary per-receipt lookups by leveraging the TableEntity list already returned by Azure Table filtering. This reduces redundant operations and makes the retrieval flow more efficient.

The endpoint was updated to enforce a controlled and configurable date window.

How Has This Been Tested?

  • Updated and executed unit tests.
  • Performed manual API checks in UAT for receipt retrieval
  • Runned succesfully integration test in UAT env

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@alessio-acitelli alessio-acitelli changed the title Pidm 1340 receipt recovery FIX: [PIDM-1340] - Receipt Recovery Mar 24, 2026
@github-actions
Copy link
Copy Markdown

This PR exceeds the recommended size of 400 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions
Copy link
Copy Markdown

Comment this PR with update_code to update openapi.json and format the code. Consider to use pre-commit to format the code.

@sonarqubecloud
Copy link
Copy Markdown

@cap-ang cap-ang marked this pull request as ready for review March 24, 2026 13:06
@cap-ang cap-ang requested a review from a team as a code owner March 24, 2026 13:06
@FedericoRuzzier FedericoRuzzier merged commit 59bafa1 into main Mar 24, 2026
14 checks passed
@FedericoRuzzier FedericoRuzzier deleted the PIDM-1340-receipt-recovery branch March 24, 2026 13:16
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.

4 participants