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

Add Support for null topic in type defs #1499

Closed
wants to merge 1 commit into from

Conversation

deanshelton913
Copy link

Addresses #1434

@deanshelton913 deanshelton913 changed the title Add Support for null topics in bloom filters Add Support for null topic in type defs Apr 23, 2021
@deanshelton913
Copy link
Author

@ricmoo Can you take a look at this when you get a sec? My IDE is throwing shade. I hate needing to force-type my null as strings. :P

@ricmoo ricmoo added investigate Under investigation and may be a bug. minor-bump Planned for the next minor version bump. on-deck This Enhancement or Bug is currently being worked on. labels Apr 23, 2021
@ricmoo
Copy link
Member

ricmoo commented Apr 23, 2021

I'm working on the changes for the next minor bump now. I'll look into this while preparing it.

@ricmoo ricmoo added enhancement New feature or improvement. and removed investigate Under investigation and may be a bug. labels Apr 24, 2021
@ricmoo
Copy link
Member

ricmoo commented May 20, 2021

Merged into 5.2.0.

Thanks! :)

@ricmoo ricmoo closed this May 20, 2021
@ricmoo ricmoo added fixed/complete This Bug is fixed or Enhancement is complete and published. and removed on-deck This Enhancement or Bug is currently being worked on. labels May 20, 2021
pull bot pushed a commit to shapeshift/ethers.js that referenced this pull request Jun 4, 2021
pull bot pushed a commit to shapeshift/ethers.js that referenced this pull request Jun 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or improvement. fixed/complete This Bug is fixed or Enhancement is complete and published. minor-bump Planned for the next minor version bump.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants