docs: Return status and logsBloom in tx receipt. Remove root attribute. Minor wording fixes.

[shanefontaine]

What was wrong?

Docs for tx receipt need updating to reflect EIP-658.

How was it fixed?

Added status and logsBloom to Eth.getTransactionReceipt() and Eth. waitForTransactionReceipt(). Removed deprecated root.

Cute Animal Picture

[pipermerriam]

This error seems like it could be caught were these code blocks to be doctested. Is that something you'd be up for adding? It should be reasonably simple, though it will require setting up a bit of chain state in a .. testsetup block.

[shanefontaine]

@pipermerriam when you get a chance, can you take a look at 72d4098. I believe that does what we want. I didn't use a .. testsetup block, so I am not sure I accomplished exactly what you want.

I skipped the "not yet mined" line in the doctest on 383ee34 because I don't think it is possible to always have a pending tx to pull from for the docs. It was giving me:

**********************************************************************
File "web3.eth.rst", line 419, in default
Failed example:
    w3.eth.getTransactionReceipt('0xf5fb3ce9c0fce91dd3d112c2ab63f9d1800f6f70667d0c4481fd72d573e12345')  # not yet mined
Expected:
    None
    # wait for it to be mined....
Got nothing
**********************************************************************

It is passing locally but CircleCI is failing here with web3.exceptions.UnhandledRequest: No providers responded to the RPC reque. This is weird, because the web3 setup seems identical to the rest of the docs. Any idea there?

[pacrob]

Closing due to age, docs have been updated since.