Skip to content

DependsOnNullOrZero bug fix#185

Open
xUJYx wants to merge 3 commits intoepartment:masterfrom
xUJYx:dependsOnNullOrZero_fix
Open

DependsOnNullOrZero bug fix#185
xUJYx wants to merge 3 commits intoepartment:masterfrom
xUJYx:dependsOnNullOrZero_fix

Conversation

@xUJYx
Copy link
Copy Markdown

@xUJYx xUJYx commented Jul 19, 2021

Fixed check for dependsOnNullOrZero to work properly (error when picking undefined or null)
http://dl3.joxi.net/drive/2021/07/19/0036/0412/2367900/00/2b12237527.jpg
https://codesandbox.io/s/eloquent-pascal-vx967?file=/src/index.js

@Tarpsvo
Copy link
Copy Markdown

Tarpsvo commented Sep 7, 2021

When making a PR, try not to format the whole file. It's impossible to see what's been changed. :)

@xUJYx
Copy link
Copy Markdown
Author

xUJYx commented Sep 7, 2021

When making a PR, try not to format the whole file. It's impossible to see what's been changed. :)

Sorry for this. As I see - it`s really hard to look through code.
Am I must to make "re-code" all the stuff? Or you can understand the diff?

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