-
Notifications
You must be signed in to change notification settings - Fork 1
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
[PAGOPA-2446] refactoring: Update status on-the-fly #273
Conversation
|
src/main/java/it/gov/pagopa/debtposition/service/payments/PaymentsService.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/service/pd/actions/PaymentPositionActionsService.java
Outdated
Show resolved
Hide resolved
…tPositionStatus.java Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
src/main/java/it/gov/pagopa/debtposition/model/enumeration/DebtPositionStatus.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/model/enumeration/DebtPositionStatus.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/model/enumeration/DebtPositionStatus.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/model/enumeration/DebtPositionStatus.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/model/enumeration/DebtPositionStatus.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/model/enumeration/DebtPositionStatus.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/model/enumeration/DebtPositionStatus.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/model/enumeration/DebtPositionStatus.java
Show resolved
Hide resolved
[PAGOPA-2446]
src/main/java/it/gov/pagopa/debtposition/service/payments/PaymentsService.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/service/payments/PaymentsService.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/service/payments/PaymentsService.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/service/payments/PaymentsService.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/service/payments/PaymentsService.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/service/payments/PaymentsService.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/service/payments/PaymentsService.java
Show resolved
Hide resolved
src/main/java/it/gov/pagopa/debtposition/service/payments/PaymentsService.java
Show resolved
Hide resolved
This pull request does not contain a valid label. Please add one of the following labels: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great work 🚀
[PAGOPA-2446]
[PAGOPA-2446]
[PAGOPA-2446]
[PAGOPA-2446]
|
List of Changes
TL;DR
validityCheckAndUpdate
method has been added 🆕 -> It is used when Get and Pay Payment OptionexpirationCheckAndUpdate
method has been added 🆕 -> It is used only when Get Payment OptionCheck on expiration or validity on report API has been removed 🗑️
These scenarios were refactored:
[GET PO by NAV scenario]
[PAY scenario]
[INVALIDATE scenario]
Motivation and Context
Refactoring status retrieve on the fly handling. Cron granularity differs from
validityDate
anddueDate
granularity. These changes helps to update status on-the-fly.How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: