-
Notifications
You must be signed in to change notification settings - Fork 26
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
fix: issue with paging account history #143
Conversation
```python | ||
from ape import chain | ||
|
||
history = chain.history["vitalik.eth"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
requires this fix: ApeWorX/ape#2170
@@ -456,10 +459,11 @@ def _get_page_of_normal_transactions( | |||
} | |||
result = self._get(params=params) | |||
|
|||
if not isinstance(result.value, list): | |||
raise UnhandledResultError(result, result.value) | |||
value = result.value or [] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
another fix: sometimes this is None
and that is still 200 / ok.
for example, give it the ZERO_ADDRESS and you get this.
@@ -278,6 +278,10 @@ def get_url_f(testnet: bool = False, tld: str = "io"): | |||
"gnosis": { | |||
"mainnet": url("gnosisscan"), | |||
}, | |||
"scroll": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was a miss from #137
What I did
the pager for account history was broken.
you would get to the end and it would try the next page anyway, causing a 500 error.
How I did it
if no more items, break out of the loop
How to verify it
Checklist