feat: add fields to tx apis, small cleanups

[zachdaniel]

#1862

Changelog

Enhancements

• Add logIndex to the tokentx action in the account module

• Add transactionHash and index to the txlistinternal action in the account module

• Clean up some unused variables/aliases in test.

• Remove a log that was emitted unnecessarily and often in tests.

  • I added an entry to CHANGELOG.md with this PR
  • If I added new functionality, I added tests covering it.
  • If I fixed a bug, I added a regression test to prevent the bug from silently reappearing again.
  • I checked whether I should update the docs and did so if necessary

[coveralls]

Pull Request Test Coverage Report for Build ac935497-f755-4a7d-beae-ef226d8b6ef9

• 3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage decreased (-0.01%) to 80.776%

---

Files with Coverage Reduction	New Missed Lines	%
apps/block_scout_web/lib/block_scout_web/channels/exchange_rate_channel.ex	1	50.0%

Totals
Change from base Build 08ec87f4-ee92-4ee3-ab52-07733d932cdb:	-0.01%
Covered Lines:	4811
Relevant Lines:	5956

---

💛 - Coveralls

[vbaranov]

/api?module=account&action=txlistinternal&address=0x1B3858E01CCE5d0491f3c96A9996a65607f3493b&offset=10

returns me 504 error, but I doubt it is because of changes from this PR. It returns 504 in prod as well.

[zachdaniel]

I can look into the performance of this endpoint for addresses with many transactions.

[vbaranov]

@zachdaniel

I can look into the performance of this endpoint for addresses with many transactions.

It would be awesome