fix: required fields for transactions and receipts

[banteg]

Motivation

this reverts incorrect changes introduced in #523

in transaction info (for example eth_getTransactionByHash), these fields are required

required:
  - blockHash
  - blockNumber
  - from
  - hash
  - transactionIndex

in transaction receipts, there fields are required.

required:
  - blockHash
  - blockNumber
  - from
  - cumulativeGasUsed
  - gasUsed
  - logs
  - logsBloom
  - transactionHash
  - transactionIndex
  - effectiveGasPrice

moreover, the original justification is also incorrect, as the author has misunderstood how geth does the serialization. this screencap is freshly out of geth --dev. as you can see, the optional fields are present, and the linked code hasn't changed since 2019.

Solution

i removed skip_serializing_if for required fields.

PR Checklist

• Added Tests
• Added Documentation
• Breaking changes

[prestwich]

if the fields are required, then removing the skip_serializing_if is not sufficient. Need to also make the prop not an Option type

@banteg

[banteg]

the keys are required in the payload, but the values can indeed be None, so the Option type seems appropriate