fix: Make props of ConnectionState optional

[cshaa]

The basics

• I branched from develop
• My pull request is against develop
• My code follows the style guide
• I ran npm run format and npm run lint

The details

Proposed Changes

Makes props in ConnectionState optional, so that the code in documentation works.

Behavior Before Change

Previously, one had to specify both block and shadow and write undefined into one of them:

{
  "kind": "flyoutToolbox",
  "contents": [
    {
      "kind": "block",
      "type": "controls_for",
      "inputs": {
        "FROM": {
          "block": {
            "type": "math_number",
            "fields": {
              "NUM": 1
            },
            "shadow": undefined, // error if omitted
          }
        },
        // ...
      }
    },
  ]
}

Behavior After Change

Now we can use the code written in the documentation:

{
  "kind": "flyoutToolbox",
  "contents": [
    {
      "kind": "block",
      "type": "controls_for",
      "inputs": {
        "FROM": {
          "block": {
            "type": "math_number",
            "fields": {
              "NUM": 1
            }
          }
        },
        // ...
      }
    },
  ]
}

Reason for Changes

This way it is both more convenient and matches the documentation better.

[maribethb]

Thank you for the PR!

[conventional-commit-lint-gcf]

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/