Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

fix(admin-api): delete published version on unpublish action #214

Merged
merged 5 commits into from
Dec 5, 2023
Merged

fix(admin-api): delete published version on unpublish action #214

merged 5 commits into from
Dec 5, 2023

Conversation

alexrodfe
Copy link
Contributor

Description

Motivation and Context

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)
  • Other changes (ci configuration, documentation or any other kind of changes)

Checklist:

  • I have created tests for my code changes, and the tests are passing.
  • I have executed the pre-commit hooks locally.
  • My change requires a change to the documentation (create a new issue if the documentation has not been updated).
  • I have updated the documentation accordingly.

@github-actions github-actions bot added the size/s label Dec 4, 2023
dfinteligenz
dfinteligenz previously approved these changes Dec 4, 2023
View reviewed changes
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Dec 5, 2023

[KAI - Admin API] 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

@alexrodfe alexrodfe merged commit a9a421e into develop Dec 5, 2023
12 checks passed
@alexrodfe alexrodfe deleted the fix/delete-published-version branch December 5, 2023 10:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@JYisus JYisus JYisus approved these changes

@dfinteligenz dfinteligenz Awaiting requested review from dfinteligenz

Assignees
No one assigned
Labels
size/s
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@alexrodfe @JYisus @dfinteligenz