Remove the unnecessary chainId parameter (#5888)

[jdevcs]

Description

Created this PR for changelog update and e2ewin tests of #5888

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.
• My changes generate no new warnings.
• Any dependent changes have been merged and published in downstream modules.
• I ran npm run lint with success and extended the tests and types if necessary.
• I ran npm run test:unit with success.
• I ran npm run test:coverage 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 checked the Deploy Preview and it looks correct.
• I have updated the CHANGELOG.md file in the root folder.
• I have linked Issue(s) with this PR in "Linked Issues" menu.

[github-actions]

Pull Request Test Coverage Report for Build 4860810584

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 75.139%

---

Totals
Change from base Build 4858743636:	0.0%
Covered Lines:	3353
Relevant Lines:	4212

---

💛 - Coveralls

[jdevcs]

All code changes in this PR are already approved in #5888 so merging in 1.x , Linting tests are not passing but its discussed in team meeting that we will go ahead with it as it will be fixed in next release auto ( due to cyclic deps )