Fix 'play' command requiring 'offset' field (#1418)

* Fix 'play' command requiring 'offset' field

* Derive 'Default' for SkipTo

CHANGELOG.md

@@ -28,6 +28,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   on Android platform.
 - [core] Fix "Invalid Credentials" when using a Keymaster access token and
   client ID on Android platform.
+= [connect] Fix "play" command not handled if missing "offset" property
 
 ### Removed
 

connect/src/spirc.rs

@@ -1040,7 +1040,7 @@ impl SpircTask {
                         context_uri: play.context.uri.clone(),
                         start_playing: true,
                         seek_to: play.options.seek_to.unwrap_or_default(),
-                        playing_track: play.options.skip_to.into(),
+                        playing_track: play.options.skip_to.unwrap_or_default().into(),
                         shuffle,
                         repeat,
                         repeat_track,

core/src/dealer/protocol/request.rs

@@ -170,7 +170,7 @@ pub struct TransferOptions {
 
 #[derive(Clone, Debug, Deserialize)]
 pub struct PlayOptions {
-    pub skip_to: SkipTo,
+    pub skip_to: Option<SkipTo>,
     #[serde(default, deserialize_with = "option_json_proto")]
     pub player_options_override: Option<ContextPlayerOptionOverrides>,
     pub license: Option<String>,
@@ -191,7 +191,7 @@ pub struct OptionsOptions {
     system_initiated: bool,
 }
 
-#[derive(Clone, Debug, Deserialize)]
+#[derive(Clone, Debug, Deserialize, Default)]
 pub struct SkipTo {
     pub track_uid: Option<String>,
     pub track_uri: Option<String>,