Skip to content

Improve resource deletion flow #66

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

Merged
merged 1 commit into from
Nov 23, 2021
Merged

Improve resource deletion flow #66

merged 1 commit into from
Nov 23, 2021

Conversation

polldo
Copy link
Contributor

@polldo polldo commented Nov 22, 2021

Motivation

I forgot about #54 (comment)
we want to improve the readability of the flow
also this change allows a more easy introduction of bulk device deletion (still not requested)

Change description

improve readability

Additional Notes

Reviewer checklist

  • PR address a single concern.
  • PR title and description are properly filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.
  • History is clean, commit messages are meaningful (see CONTRIBUTING.md) and are well formatted.

Co-authored-by: Giuseppe Lumia <g.lumia@outlook.com>
@polldo polldo force-pushed the polldo/improve-delete branch from da94a57 to e1a8469 Compare November 22, 2021 16:14
@polldo polldo requested a review from glumia November 22, 2021 16:14
@polldo polldo merged commit 68a4aa2 into main Nov 23, 2021
@polldo polldo deleted the polldo/improve-delete branch November 23, 2021 08:48
polldo pushed a commit that referenced this pull request Sep 2, 2022
Co-authored-by: Giuseppe Lumia <g.lumia@outlook.com>
polldo pushed a commit that referenced this pull request Sep 2, 2022
Co-authored-by: Giuseppe Lumia <g.lumia@outlook.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