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.

Sign up for GitHub

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

Jump to bottom

[FEATURE REQUEST] make onionjuggler-tui case statements simpler #74

Open
nyxnor opened this issue Aug 17, 2022 · 1 comment
Open

[FEATURE REQUEST] make onionjuggler-tui case statements simpler #74

nyxnor opened this issue Aug 17, 2022 · 1 comment
Labels
bug Something isn't working enhancement New feature or request

Comments

@nyxnor
Copy link
Owner

nyxnor commented Aug 17, 2022

Instead of comparing is variable is not empty -n in a lot of if else, call a function and return 1 from inside if the variable is empty, if not, continue the steps without needing tons of if statements.
@nyxnor nyxnor added bug Something isn't working enhancement New feature or request labels Aug 17, 2022
@nyxnor nyxnor changed the title [BUG] make onionjuggler-tui case statements simpler [FEATURE REQUEST] make onionjuggler-tui case statements simpler Aug 20, 2022
@nyxnor
Copy link
Owner Author

nyxnor commented Aug 20, 2022

This is code cleaning, an enhancement, not a bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nyxnor