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

skywire-cli terminal user interface improvements #1390

Merged
merged 17 commits into from
Oct 22, 2022
Merged

skywire-cli terminal user interface improvements #1390

merged 17 commits into from
Oct 22, 2022

Conversation

0pcom
Copy link
Collaborator

@0pcom 0pcom commented Oct 12, 2022

Fixes #1385

Changes:

How to test this PR:

@0pcom 0pcom changed the title skywire-cli terminal user interface improvements skywire-cli terminal user interface improvements [WIP] Oct 12, 2022
0pcom and others added 9 commits October 13, 2022 11:48
…lobal flag to vpn and visor subcommands where it is actually used. Hide json global flag. Remove the last line of the help menu 'for more information about a command'
… rule rm --all -s flag to remove all transports / routing rules. Not fully implemented. make format check
@0pcom
Copy link
Collaborator Author

0pcom commented Oct 17, 2022

Before
image

After
image

Before
image

After
image

Before
image

After
image

Before
image

After
image

Before
image

After
image

Before
image

After
image

Before
image

After
image

Before
image

After
image

Before
image

After
image

Before

After
image

Before
image

After
image

Ready with this one, now

I've left some things commented out which can be implemented after this is merged

@0pcom 0pcom changed the title skywire-cli terminal user interface improvements [WIP] skywire-cli terminal user interface improvements Oct 19, 2022
Copy link
Contributor

@mrpalide mrpalide left a comment

Choose a reason for hiding this comment

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

Approved, just small comment. Will merge after your decision about it.

Comment on lines 340 to 345
if logLvl != "" {
if logLvl == "INFO" || logLvl == "DEBUG" || logLvl == "TRACE" {
log.Info("setting log level to: ", logLvl)
conf.LogLevel = logLvl
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Be good if the behave of this condition not limited to capital words, means set info and Info works like INFO. Currently just INFO and TRACE and DEBUG work well.

@0pcom
Copy link
Collaborator Author

0pcom commented Oct 22, 2022

Before

image
image

After
image
image

@0pcom 0pcom merged commit 70cfa53 into skycoin:develop Oct 22, 2022
@0pcom 0pcom deleted the fix/skywire-cli-tui branch October 22, 2022 01:14
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