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

UX Changes in litmusctl using promptui library. #156

Merged
merged 6 commits into from
Nov 15, 2023

Conversation

deep-poharkar
Copy link
Contributor

@deep-poharkar deep-poharkar commented Sep 27, 2023

UX Changes In litmusctl

  1. Config

setAccount : Added Interactive Prompts and Password Input Masking
set-acc

useAccount : confirmation message
conf

  1. Delete

Used the PromptUI library to ask the user to enter the Project ID and Chaos Experiment ID so that it eliminates the need for manual checks and provides a more user-friendly experience. Also, asks the user "Are you sure you want to delete this Chaos Experiment? (y/n)," allowing the user to confirm or cancel the deletion.

del
  1. Get

Project and Experiment : Allows the user to select an output format (json, yaml, table) using “promptui.Select”. Also, implemented pagination inside get experiments so that we can get experiments in pages where the user can enter q to quit,
if outputFormat is table.

get-ex format
  1. Describe

Prompt to select output format as either YAML or JSON.
output-prompt

  1. Create

Prompting the user to enter a project name and showing Error and confirmation message.

create-err
  1. Flags

Added experiment ID and experiment-run ID flags inside get.

pkg/cmd/create/project.go Outdated Show resolved Hide resolved
pkg/cmd/delete/experiment.go Outdated Show resolved Hide resolved
pkg/cmd/get/experiments.go Outdated Show resolved Hide resolved
pkg/cmd/get/experiments.go Outdated Show resolved Hide resolved
pkg/cmd/get/projects.go Outdated Show resolved Hide resolved
pkg/cmd/create/project.go Outdated Show resolved Hide resolved
pkg/cmd/describe/experiment.go Show resolved Hide resolved
@Nageshbansal
Copy link
Collaborator

LGTM 🚀

Usage_0.23.0.md Outdated Show resolved Hide resolved
pkg/cmd/create/experiment.go Outdated Show resolved Hide resolved
Signed-off-by: deep-poharkar <[email protected]>
Copy link
Collaborator

@Saranya-jena Saranya-jena left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@SarthakJain26 SarthakJain26 merged commit 6b68c4d into litmuschaos:master Nov 15, 2023
4 checks passed
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.

4 participants