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(core): Update Resource Mapping delete to use get before delete for cli output #398

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

ryanulit
Copy link
Contributor

@ryanulit ryanulit commented Oct 2, 2024

Due to changes made in opentdf/platform#1422 to only return an object with the ID for delete operations, we need to update the otdfctl Resource Mapping delete operation to make a Get request from the DB before deletion of the record. The object returned from the Get is then used for the user-friendly CLI output of the whole object.

@ryanulit ryanulit requested a review from a team as a code owner October 2, 2024 20:12
@ryanulit ryanulit merged commit 79f2a42 into main Oct 2, 2024
6 checks passed
@ryanulit ryanulit deleted the feat/1421-use-get-for-cli-output-after-delete-rm branch October 2, 2024 20:18
jakedoublev pushed a commit that referenced this pull request Oct 15, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.15.0](v0.14.0...v0.15.0)
(2024-10-15)


### Features

* **core:** DSP-51 - deprecate PublicKey local field
([#400](#400))
([1955800](1955800))
* **core:** Update Resource Mapping delete to use get before delete for
cli output ([#398](#398))
([79f2a42](79f2a42))


### Bug Fixes

* **core:** build with latest opentdf releases
([#404](#404))
([969b82b](969b82b))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: opentdf-automation[bot] <149537512+opentdf-automation[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants