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

Eliminate nested ternary operators #57

Closed
vojtechsimetka opened this issue Apr 6, 2021 · 0 comments · Fixed by #181
Closed

Eliminate nested ternary operators #57

vojtechsimetka opened this issue Apr 6, 2021 · 0 comments · Fixed by #181
Labels
type:issue
Milestone
web app improvements

Comments

@vojtechsimetka
Copy link
Contributor

Right now there is a number of nested ternary operators in several components. All of them have a // FIXME comment and should be removed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type:issue
Projects
None yet
Milestone
web app improvements
Development

Successfully merging a pull request may close this issue.

fix: remove nested ternary operator and simplify ping peer functionality ethersphere/bee-dashboard
1 participant
@vojtechsimetka