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: Add auth addr to transactions in responses. #1459

Merged
merged 1 commit into from
Feb 8, 2023

Conversation

winder
Copy link
Contributor

@winder winder commented Feb 5, 2023

Summary

Add missing field to transaction response.

@codecov
Copy link

codecov bot commented Feb 5, 2023

Codecov Report

Merging #1459 (b555a9c) into develop (323ff96) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop    #1459      +/-   ##
===========================================
+ Coverage    65.39%   65.42%   +0.02%     
===========================================
  Files           83       83              
  Lines        11411    11412       +1     
===========================================
+ Hits          7462     7466       +4     
+ Misses        3379     3377       -2     
+ Partials       570      569       -1     
Impacted Files Coverage Δ
api/converter_utils.go 92.66% <100.00%> (+0.01%) ⬆️
fetcher/fetcher.go 63.98% <0.00%> (+1.27%) ⬆️

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

@winder winder changed the title Add auth addr to transaction response. API: Add auth addr to transactions in responses. Feb 6, 2023
@winder winder added the Bug-Fix label Feb 6, 2023
@shiqizng shiqizng merged commit 3e3d479 into algorand:develop Feb 8, 2023
@winder winder deleted the will/auth-addr-fix branch March 8, 2023 13:50
winder added a commit to winder/indexer that referenced this pull request Mar 13, 2023
@winder winder mentioned this pull request Mar 13, 2023
@winder winder mentioned this pull request Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants