Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IOCOM-1952] substituted payment updater api with pagopa ecommerce api #312

Merged
merged 16 commits into from
Jan 21, 2025

Conversation

acuiuli
Copy link
Contributor

@acuiuli acuiuli commented Nov 29, 2024

List of Changes

Motivation and Context

The payment updater api in the GetMessage handler that checks if a message has been paid is not working correctly and it often gives errors. This api has been substituted with the pagopa ecommerce api that directly checks the payment, this should avoid unpredictable errors from the PaymentUpdater service.

How Has This Been Tested?

  • local unit test

Screenshots (if appropriate):

Types of changes

  • Chore (nothing changes by a user perspective)
  • 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.

Copy link

github-actions bot commented Nov 29, 2024

Jira Pull Request Link

This Pull Request refers to the following Jira issue IOCOM-1952

@acuiuli acuiuli marked this pull request as ready for review January 16, 2025 10:48
@acuiuli acuiuli requested a review from a team as a code owner January 16, 2025 10:48
@acuiuli acuiuli merged commit acb693f into master Jan 21, 2025
6 checks passed
@acuiuli acuiuli deleted the IOCOM-1952-add-pagopa-ecommerce branch January 21, 2025 08:44
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