Nikos/5821/transaction type fix

[nikoulai]

Description

closes #5821

Checklist for 1.x:

• I have selected the correct base branch.
• I have performed a self-review of my own code.
• I have commented my code, particularly in hard-to-understand areas.
• I have made corresponding changes to the documentation.
• My changes generate no new warnings.
• Any dependent changes have been merged and published in downstream modules.
• I ran npm run dtslint with success and extended the tests and types if necessary.
• I ran npm run test:cov and my test cases cover all the lines and branches of the added code.
• I ran npm run build with success.
• I have tested the built dist/web3.min.js in a browser.
• I have tested my code on the live network.
• I have checked the Deploy Preview and it looks correct.
• I have updated the CHANGELOG.md file in the root folder.

[coveralls]

Pull Request Test Coverage Report for Build 4292828794

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 271 unchanged lines in 7 files lost coverage.
• Overall coverage increased (+2.4%) to 75.018%

---

Files with Coverage Reduction	New Missed Lines	%
packages/web3-core-requestmanager/src/jsonrpc.js	2	64.29%
packages/web3-core-helpers/src/formatters.js	9	83.86%
packages/web3-core-helpers/src/errors.js	29	1.56%
packages/web3-utils/src/soliditySha3.js	34	3.43%
packages/web3-utils/src/index.js	40	40.3%
packages/web3-utils/src/utils.js	46	10.39%
packages/web3-eth-accounts/src/index.js	111	22.94%

Totals
Change from base Build 4279822756:	2.4%
Covered Lines:	3310
Relevant Lines:	4161

---

💛 - Coveralls

[jdevcs]

approved, and re-triggered build tests, seems these were failing due to time out,.