Add bucket deletion support in the OSCAR TUI by wiring the existing delete flow to Buckets view#76
Open
Add bucket deletion support in the OSCAR TUI by wiring the existing delete flow to Buckets view#76
Conversation
* Included support for oscar-cli cluster status (#67) * Included support for /system/status oscar-cli cluster status * status update * update readme --------- Co-authored-by: SergioLangaritaBenitez <sergiolangaritabenitez@gmail.com> * Include support for cluster status in the TUI (#68) * Included support for /system/status oscar-cli cluster status * Included support for /system/status in the TUI * Included support for cluster status in the tui via oscar-cli interactive --------- Co-authored-by: SergioLangaritaBenitez <sergiolangaritabenitez@gmail.com> * Refactored TUI & Included Search Functionality (#70) * Splitted TUI Split the TUI into focused files so pkg/tui/app.go is now mostly the UI wiring/entrypoint while helpers live alongside it: + pkg/tui/state.go holds constants/enums and uiState/bucket object structs; formatters.go and table_helpers.go hold pure helpers. + View/controller logic now lives in services_view.go, buckets_view.go, and bucket_objects.go; + cross-cutting UI helpers went to dialogs.go, search.go, and auto_refresh.go. + app.go cleaned up accordingly, with redundant definitions removed and imports trimmed. * Improved search functionality + Implemented support for search via "/" across Services, Buckets, Clusters and Details panels, including substrings. * resolve conflict * resolve conflict --------- Co-authored-by: SergioLangaritaBenitez <sergiolangaritabenitez@gmail.com> * Improved Log Management in the TUI (#71) * Splitted TUI Split the TUI into focused files so pkg/tui/app.go is now mostly the UI wiring/entrypoint while helpers live alongside it: + pkg/tui/state.go holds constants/enums and uiState/bucket object structs; formatters.go and table_helpers.go hold pure helpers. + View/controller logic now lives in services_view.go, buckets_view.go, and bucket_objects.go; + cross-cutting UI helpers went to dialogs.go, search.go, and auto_refresh.go. + app.go cleaned up accordingly, with redundant definitions removed and imports trimmed. * Improved search functionality + Implemented support for search via "/" across Services, Buckets, Clusters and Details panels, including substrings. * Improved log management in the TUI --------- Co-authored-by: SergioLangaritaBenitez <sergiolangaritabenitez@gmail.com> * fix invocation (#73) * Allow to override bucket name to match service name (#72) This is applied both for oscar-cli hub and oscar-cli apply --------- Co-authored-by: Estíbaliz Parcero <5576714+esparig@users.noreply.github.com> Co-authored-by: Germán Moltó <gmolto@dsic.upv.es>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What Changed
Key Binding
Both now delete the selected item in the current table context:
Files Updated
Validation
Impact
guidance.