Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update index.d.ts #4955

Merged
merged 5 commits into from
Jul 18, 2022
Merged

Update index.d.ts #4955

merged 5 commits into from
Jul 18, 2022

Conversation

bigbizze
Copy link

the changed callback type returns an array of event data rather than an object of one event's data: https://www.quicknode.com/docs/ethereum/eth_getLogs

this callback returns an array of event data
@jdevcs jdevcs added 1.x 1.0 related issues Review Needed Maintainer(s) need to review labels Apr 22, 2022
Copy link
Contributor

@jdevcs jdevcs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bigbizze changelog update is missing.

Copy link
Contributor

@nazarhussain nazarhussain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bigbizze Please add details to the CHANGELOG.md rest seems good.

@coveralls
Copy link

coveralls commented Apr 28, 2022

Pull Request Test Coverage Report for Build 2338930161

  • 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 72.211%

Totals Coverage Status
Change from base Build 2337627991: 0.0%
Covered Lines: 3372
Relevant Lines: 4401

💛 - Coveralls

@avkos avkos removed their request for review May 3, 2022 07:10
@nazarhussain
Copy link
Contributor

@bigbizze Did you get chance to look into this PR. Please add the CHANGELOG.md entry so we can get this PR ready to merge.

@github-actions
Copy link

This PR has been automatically marked as stale beacause it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. If you believe this was a mistake, please comment.

@github-actions github-actions bot added the Stale Has not received enough activity label Jul 17, 2022
@nazarhussain nazarhussain changed the base branch from 1.x to nazarhussain-patch-1 July 18, 2022 08:40
@nazarhussain nazarhussain merged commit 32fa296 into web3:nazarhussain-patch-1 Jul 18, 2022
@nazarhussain nazarhussain mentioned this pull request Jul 18, 2022
16 tasks
avkos added a commit that referenced this pull request Sep 20, 2022
* Update CHANGELOG.md

* Update index.d.ts (#4955)

this callback returns an array of event data

Co-authored-by: Oleksii Kosynskyi <oleksii.kosynskyi@gmail.com>
Co-authored-by: Nazar Hussain <nazarhussain@gmail.com>
Co-authored-by: Nikos Iliakis <nikoulai@users.noreply.github.com>

* Update CHANGELOG.md

* Update CHANGELOG.md

Co-authored-by: Junaid <86780488+jdevcs@users.noreply.github.com>

* Apply suggestions from code review

Co-authored-by: Muhammad Altabba <24407834+Muhammad-Altabba@users.noreply.github.com>

* fix CHANGELOG

Co-authored-by: bigbizze <42508203+bigbizze@users.noreply.github.com>
Co-authored-by: Oleksii Kosynskyi <oleksii.kosynskyi@gmail.com>
Co-authored-by: Nikos Iliakis <nikoulai@users.noreply.github.com>
Co-authored-by: Junaid <86780488+jdevcs@users.noreply.github.com>
Co-authored-by: Muhammad Altabba <24407834+Muhammad-Altabba@users.noreply.github.com>
Co-authored-by: Alex <alex.luu@mail.utoronto.ca>
@jdevcs jdevcs mentioned this pull request Oct 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.x 1.0 related issues Review Needed Maintainer(s) need to review Stale Has not received enough activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants