Skip to content

feat: display memo with ascii encoding in addition to hex#2778

Merged
heidar-stacks merged 1 commit intomainfrom
feat/display-memo-ascii-encoding
Apr 7, 2026
Merged

feat: display memo with ascii encoding in addition to hex#2778
heidar-stacks merged 1 commit intomainfrom
feat/display-memo-ascii-encoding

Conversation

@heidar-stacks
Copy link
Copy Markdown
Contributor

@heidar-stacks heidar-stacks commented Apr 7, 2026

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

display memo with ascii encoding as well as hexadecimal encoding on token transfer transactions

Issue ticket number and link

closes #2770

Checklist:

  • I have performed a self-review of my code.
  • I have tested the change on desktop and mobile.
  • I have added thorough tests where applicable.
  • I've added analytics and error logging where applicable.

Screenshots (if appropriate):

Marketing Summary

Transaction memos are now displayed in human-readable text alongside the raw hex, making it easier to verify exchange deposit IDs and other memo values at a glance.

@github-project-automation github-project-automation bot moved this to 🆕 New in Stacks Explorer Apr 7, 2026
Copilot AI review requested due to automatic review settings April 7, 2026 15:41
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
stacks-labs-explorer Ready Ready Preview, Comment Apr 7, 2026 3:45pm

Request Review

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
[...Id]/redesign/tx-summary/TokenTransferTxSummary.tsx](https://app.codecov.io/gh/stx-labs/explorer/pull/2778?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=stx-labs#diff-c3JjL2FwcC90eGlkL1t0eElkXS9yZWRlc2lnbi90eC1zdW1tYXJ5L1Rva2VuVHJhbnNmZXJUeFN1bW1hcnkudHN4) 0.00% 7 Missing ⚠️

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a feature to display transaction memos with ASCII encoding in addition to hexadecimal encoding, addressing issue #2770. The feature makes it easier for users to verify exchange deposit IDs and other memo values at a glance by showing human-readable text alongside the raw hex representation.

Changes:

  • Added ASCII decoding of transaction memos using the existing getMemoString utility
  • Enhanced the Memo display in token transfer transactions to show both decoded ASCII (when available) and the original hex value
  • Added necessary UI component imports (Text and Stack) for the new memo display layout

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/app/txid/[txId]/redesign/tx-summary/TokenTransferTxSummary.tsx
@heidar-stacks heidar-stacks merged commit 441490a into main Apr 7, 2026
15 checks passed
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in Stacks Explorer Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Display memo with ASCII encoding as well as hexadecimal encoding

4 participants