Skip to content

Conversation

@ThomasCAI-mlv
Copy link
Collaborator

No description provided.

@ThomasCAI-mlv ThomasCAI-mlv added the feature This issue or pull request contains a new feature label Oct 16, 2024
@ThomasCAI-mlv ThomasCAI-mlv self-assigned this Oct 16, 2024
@ThomasCAI-mlv ThomasCAI-mlv changed the title Return list of deleted resources in deletion API Return list of deleted resources in resources deletion API Oct 16, 2024
@ThomasCAI-mlv ThomasCAI-mlv force-pushed the feat/return-deleted-resources branch 7 times, most recently from e761509 to df87581 Compare October 21, 2024 06:14
@ThomasCAI-mlv ThomasCAI-mlv force-pushed the feat/return-deleted-resources branch from df87581 to 8a3b4f1 Compare October 21, 2024 06:26
@ThomasCAI-mlv ThomasCAI-mlv reopened this Oct 21, 2024
@ThomasCAI-mlv ThomasCAI-mlv force-pushed the feat/return-deleted-resources branch 6 times, most recently from 2c63c68 to 662e773 Compare October 21, 2024 13:10
@ThomasCAI-mlv ThomasCAI-mlv force-pushed the feat/return-deleted-resources branch from 662e773 to b103b77 Compare October 21, 2024 13:20
@ThomasCAI-mlv ThomasCAI-mlv marked this pull request as ready for review October 21, 2024 13:28
Copy link
Collaborator

@loicgreffier loicgreffier left a comment

Choose a reason for hiding this comment

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

@ThomasCAI-mlv Just one comment on ConnectorIntegrationTest

@sonarqubecloud
Copy link

Copy link
Collaborator

@loicgreffier loicgreffier left a comment

Choose a reason for hiding this comment

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

LGTM

@loicgreffier loicgreffier merged commit 1d844e5 into master Oct 23, 2024
3 checks passed
@loicgreffier loicgreffier deleted the feat/return-deleted-resources branch October 23, 2024 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature This issue or pull request contains a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants