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
feat: export isDraft
function
#220
Conversation
I guess you could do |
Should isDraft return true for a new object that was added to the draft
tree? Probably not?
Op vr 19 okt. 2018 20:51 schreef Alec Larson <notifications@github.com>:
… I guess you *could* do !!original(value) to get the same effect as
isDraft(value). WDYT?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#220 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABvGhGWgridOMwKAB7C5-lNHy3oLlc4Nks5umh60gaJpZM4XiHLq>
.
|
@mweststrate "Just proxies" is the idea. LGTM? |
@aleclarson LGTM. Could you add some tests for both proxy and non proxied version? |
Are tests necessary? It's a very simple function: |
For regression against internal refactorings, yeah please provide a some
false and positive tests to protect against accidental mistakes.
Op zo 21 okt. 2018 12:34 schreef Alec Larson <notifications@github.com>:
… Are tests necessary? It's a very simple function: const isDraft = (value)
=> !!value && !!value[PROXY_STATE]
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#220 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABvGhCsXUlyDkHXpJx_qEpOYbzv_6U-Kks5unE2sgaJpZM4XiHLq>
.
|
Done! |
75d6f82
to
f2b3e4b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great!
To use in other libraries