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

Function should only do one thing - Invalid example #275

Open
atlanteh opened this issue Dec 5, 2019 · 1 comment
Open

Function should only do one thing - Invalid example #275

atlanteh opened this issue Dec 5, 2019 · 1 comment

Comments

@atlanteh
Copy link

@atlanteh atlanteh commented Dec 5, 2019

Checking the db for "isClientActive" would likely return a promise, and this makes clientRecord.isActive always return false since clientRecord is actually a Promise which doesn't have isActive prop.. But even if fix that.. When we filter by this function for active clients we will get all record since this function returns a promise which evaluates to true when filtering

@ryanmcdermott
Copy link
Owner

@ryanmcdermott ryanmcdermott commented Jan 10, 2020

Feel free to open a PR fixing this! Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.