fix(sisyfos): Use nullish check rather than or on channel reset

[PeterC89]

About the Contributor

This pull request is posted on behalf of the BBC.

Type of Contribution

Bug fix

Current Behavior

When setting a full Sisyfos channel state falsy values (E.G. a fader level of 0) will cause the default to be sent instead of the desired value.

New Behavior

The correct value is sent

Testing Instructions

• Trigger a set channel command (E.G. with triggerValue on an individual channel) with a fader level of 0 and verify that Sisyfos receives 0 as expected

Other Information

Status

• PR is ready to be reviewed.
• The functionality has been tested by the author.
• Relevant unit tests has been added / updated.
• Relevant documentation (code comments, system documentation) has been added / updated.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Project coverage is 56.60%. Comparing base (59f6d37) to head (65a73c5).

Files with missing lines	Patch %	Lines
...te-resolver/src/integrations/sisyfos/connection.ts	0.00%	5 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           release52     #353    +/-   ##
===========================================
  Coverage      56.60%   56.60%            
===========================================
  Files            131      131            
  Lines          10265    10265            
  Branches        2503     2563    +60     
===========================================
  Hits            5810     5810            
+ Misses          4453     4087   -366     
- Partials           2      368   +366     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nytamin]

Looks good! I'll cherry-pick this bug fix into release51 as well