add tx.chainId if tx.common.chainId not provided

[jdevcs]

Description

#4208

Throw error if user provided tx.chainId and tx.common.customChain.chainIdplus these are not matching.

Not considering tx.chainId if tx.common.customChain.chainId is provided for web3.eth.accounts.signTransaction function ,

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

• 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.
• I ran npm run dtslint with success and extended the tests and types if necessary.
• I ran npm run test:unit with success.
• 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 and tested dist/web3.min.js in a browser.
• I have tested my code on the live network.
• I have updated the CHANGELOG.md file in the root folder.

[render]

Your Render PR Server URL is https://web3-js-pr-4293.onrender.com.

Follow its progress at https://dashboard.render.com/static/srv-c4r37ssobjddd9dbhe50.

[coveralls]

Pull Request Test Coverage Report for Build 1261469566

• 1 of 16 (6.25%) changed or added relevant lines in 1 file are covered.
• 40 unchanged lines in 2 files lost coverage.
• Overall coverage decreased (-0.4%) to 72.548%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
packages/web3-eth-accounts/src/index.js	1	16	6.25%

Files with Coverage Reduction	New Missed Lines	%
packages/web3-eth-accounts/src/index.js	1	23.77%
packages/web3-eth-accounts/lib/index.js	39	85.36%

Totals
Change from base Build 1260518134:	-0.4%
Covered Lines:	3334
Relevant Lines:	4340

---

💛 - Coveralls

[spacesailor24]

Looks good, just a formatting suggestion