This PR fixes an inconsistency in how "empty observable warning" is issued between cases with one and many observable props. Since the warning is already issued for non-behavior observables in the RenderMany case, this fix only adds it to RenderOne case.
We will need to update the wording in the warning message since it will no longer warn only for an empty observable case.
Also I don't consider this to be a breaking change, so I'm not bumping the minor version.
[x] refactor empty warning tests
[x] fix empty warning for RenderOne
[x] improve warning message
[ ] can we add the offending property name to the warning message? Not this time
Originally reported in #25
This PR fixes an inconsistency in how "empty observable warning" is issued between cases with one and many observable props. Since the warning is already issued for non-behavior observables in the
RenderManycase, this fix only adds it toRenderOnecase.We will need to update the wording in the warning message since it will no longer warn only for an empty observable case.
Also I don't consider this to be a breaking change, so I'm not bumping the minor version.
can we add the offending property name to the warning message?Not this time