Skip to content

Conversation

svariant
Copy link
Collaborator

@svariant svariant commented Jul 2, 2025

List of Changes

  • Added flowDate param to API getAllPublishedFlows to filter flows by date
  • Updated unit tests
  • Update openapi
  • Commented google-java-format in GHA code review

Motivation and Context

PIDM-730

How Has This Been Tested?

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.

@svariant svariant requested a review from a team as a code owner July 2, 2025 12:46
@svariant svariant added the minor label Jul 2, 2025
Copy link
Contributor

github-actions bot commented Jul 2, 2025

The default action is to increase the PATCH number of SEMVER. Set IGNORE-FOR-RELEASE if you want to skip SEMVER bump. BREAKING-CHANGE and NEW-RELEASE must be run from GH Actions section manually.

@github-actions github-actions bot added the patch label Jul 2, 2025
Copy link
Contributor

github-actions bot commented Jul 2, 2025

The default action is to increase the PATCH number of SEMVER. Set IGNORE-FOR-RELEASE if you want to skip SEMVER bump. BREAKING-CHANGE and NEW-RELEASE must be run from GH Actions section manually.

@svariant svariant marked this pull request as draft July 2, 2025 12:49
@pagopa pagopa deleted a comment from github-actions bot Jul 3, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 3, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 3, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 3, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 3, 2025
@svariant svariant marked this pull request as ready for review July 3, 2025 14:20
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
@pagopa pagopa deleted a comment from github-actions bot Jul 4, 2025
Copy link

Copy link
Contributor

@andrea-deri andrea-deri left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Copy link
Contributor

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.

@pasqualespica pasqualespica merged commit 40b3eaf into main Jul 23, 2025
15 of 16 checks passed
@pasqualespica pasqualespica deleted the PIDM-730-add-flow-date-param-getAllPublishedFlows branch July 23, 2025 13:02
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.

5 participants