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

api: Fix errFailedRetrievingStateDelta response. #5452

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

winder
Copy link
Contributor

@winder winder commented Jun 5, 2023

Summary

The errFailedRetrievingStateDelta variable was being used without adding the error.

@winder winder requested review from Eric-Warehime and a team June 5, 2023 19:23
@winder winder self-assigned this Jun 5, 2023
@winder winder changed the title Fix errFailedRetrievingStateDelta response. api: Fix errFailedRetrievingStateDelta response. Jun 5, 2023
@winder winder added the Bug-Fix label Jun 5, 2023
@codecov
Copy link

codecov bot commented Jun 5, 2023

Codecov Report

Merging #5452 (ac49afa) into master (4b70342) will increase coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #5452      +/-   ##
==========================================
+ Coverage   55.45%   55.49%   +0.03%     
==========================================
  Files         447      447              
  Lines       63290    63290              
==========================================
+ Hits        35098    35120      +22     
+ Misses      25815    25798      -17     
+ Partials     2377     2372       -5     
Impacted Files Coverage Δ
daemon/algod/api/server/v2/handlers.go 0.82% <0.00%> (ø)

... and 7 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@winder winder marked this pull request as ready for review June 5, 2023 20:04
@winder winder merged commit 1994341 into algorand:master Jun 5, 2023
@winder winder deleted the will/eval-tracer-fix branch June 6, 2023 11:56
onetechnical pushed a commit to onetechnical/go-algorand that referenced this pull request Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants