Skip to content

Add block explorer link to transaction output#1998

Merged
chasefleming merged 3 commits into
masterfrom
cf/tx-link
Jul 28, 2025
Merged

Add block explorer link to transaction output#1998
chasefleming merged 3 commits into
masterfrom
cf/tx-link

Conversation

@chasefleming
Copy link
Copy Markdown
Member

@chasefleming chasefleming commented Jul 23, 2025

Closes #1791

Screenshot 2025-07-23 at 5 06 13 PM

For contributor use:

  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work
  • Code follows the standards mentioned here
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

@chasefleming chasefleming added the Improvement Technical work without new features, refactoring, improving tests label Jul 23, 2025
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 23, 2025

Codecov Report

Attention: Patch coverage is 87.50000% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/transactions/transactions.go 84.00% 4 Missing ⚠️

📢 Thoughts on this report? Let us know!

@chasefleming chasefleming merged commit f0d97be into master Jul 28, 2025
5 checks passed
@chasefleming chasefleming deleted the cf/tx-link branch July 28, 2025 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Improvement Technical work without new features, refactoring, improving tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Link transactions to Flowscan in CLI outputs

3 participants