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

feat: sviluppo afm utils Pagopa 1162 #30

Merged
merged 3 commits into from
Aug 23, 2023

Conversation

FedericoRuzzier
Copy link

List of Changes

Adding the pspBusinessName to the CDI entity and bundle request model. In addition, this field is also set up in the CDI service when creating the bundle.

This addition was required for the import of the bundle from the CDI

Motivation and Context

How Has This Been Tested?

This has been tested manually and through unit tests.

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.

@FedericoRuzzier FedericoRuzzier added enhancement New feature or request minor labels Aug 22, 2023
@FedericoRuzzier FedericoRuzzier changed the title Pagopa 1162 sviluppo afm utils feat: sviluppo afm utils Pagopa 1162 Aug 22, 2023
@FedericoRuzzier
Copy link
Author

update_code

@github-actions
Copy link

Update Code is failed. Please retry.

@github-actions
Copy link

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

@sonarcloud
Copy link

sonarcloud bot commented Aug 23, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

Copy link
Collaborator

@alessio-acitelli alessio-acitelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job 👍

@FedericoRuzzier FedericoRuzzier merged commit 0cbf7cb into main Aug 23, 2023
8 of 9 checks passed
@FedericoRuzzier FedericoRuzzier deleted the PAGOPA-1162-sviluppo-afm-utils branch August 23, 2023 15:47
@FedericoRuzzier FedericoRuzzier temporarily deployed to dev August 23, 2023 15:50 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants