fix undefined and null values

YousefED commented 2 years ago

closes https://github.com/YousefED/SyncedStore/issues/31

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 1569207236

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

7 of 7 (100.0%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 82.309%

Totals
Change from base Build 1558005461:	0.3%
Covered Lines:	497
Relevant Lines:	584

💛 - Coveralls

jasonm commented 2 years ago

@YousefED Thanks for the fix! Would you be able to issue an 0.3.5 release? I tried installing this as a git reference but am unable to get it to build: https://gist.github.com/jasonm/5aca1c39db2a194db1c97e9b5092d83f

jasonm commented 2 years ago

Ah I wonder if it's because I'm using Node v16. The GitHub Actions CI build for this repo uses v14 🤔

YousefED commented 2 years ago

@jasonm my bad, just released this!

jasonm commented 2 years ago

@YousefED no worries - thank you very much! (And happy new year!)

YousefED / SyncedStore