Skip to content

✨ app: add stellar memo#986

Merged
cruzdanilo merged 1 commit intomainfrom
memo-ui
Apr 29, 2026
Merged

✨ app: add stellar memo#986
cruzdanilo merged 1 commit intomainfrom
memo-ui

Conversation

@franm91
Copy link
Copy Markdown
Member

@franm91 franm91 commented Apr 28, 2026


Open in Devin Review

Summary by CodeRabbit

  • New Features

    • View and copy memos for cryptocurrency deposits: memos are shown when present and can be copied.
    • Sharing deposit details now includes the memo when available.
    • Copy actions display a short toast and provide haptic feedback on success.
  • Style

    • Updated copy button icon for clearer copy affordance.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 28, 2026

🦋 Changeset detected

Latest commit: 14d4e28

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 09ce1b7b-3beb-42d3-ba6f-deffa8b5a54b

📥 Commits

Reviewing files that changed from the base of the PR and between eaf4d4a and 14d4e28.

📒 Files selected for processing (2)
  • .changeset/fluffy-cameras-rush.md
  • src/components/add-funds/AddCrypto.tsx

Walkthrough

Reads an optional Stellar memo from bridge deposit results in the mobile add-funds flow, conditionally displays it, includes it when sharing deposit details, and adds a memo-copy action that writes to clipboard and triggers a native toast plus success haptic feedback.

Changes

Cohort / File(s) Summary
Changeset Metadata
.changeset/fluffy-cameras-rush.md
Adds a changeset declaring a patch bump for @exactly/mobile with the message "✨ add stellar memo".
Memo UI & Interactions
src/components/add-funds/AddCrypto.tsx
Reads optional memo from deposit result, conditionally renders a memo section, appends a labeled Memo line to the shared message, swaps address copy icon from Files to Copy, and adds memo-copy handling that writes to clipboard and shows a short native toast with success haptic feedback.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • cruzdanilo
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title '✨ app: add stellar memo' directly summarizes the main change—adding stellar memo functionality to the app component, which matches the core functionality described in the changeset and AddCrypto.tsx modifications.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch memo-ui

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

chatgpt-codex-connector[bot]

This comment was marked as resolved.

gemini-code-assist[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@sentry
Copy link
Copy Markdown

sentry Bot commented Apr 28, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.82%. Comparing base (acef232) to head (14d4e28).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #986      +/-   ##
==========================================
- Coverage   72.90%   72.82%   -0.09%     
==========================================
  Files         229      229              
  Lines        8612     8585      -27     
  Branches     2802     2786      -16     
==========================================
- Hits         6279     6252      -27     
+ Misses       2104     2103       -1     
- Partials      229      230       +1     
Flag Coverage Δ
e2e 72.82% <ø> (+1.67%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

chatgpt-codex-connector[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 14d4e28f96

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread src/components/add-funds/AddCrypto.tsx
@cruzdanilo cruzdanilo merged commit 14d4e28 into main Apr 29, 2026
15 of 17 checks passed
@cruzdanilo cruzdanilo deleted the memo-ui branch April 29, 2026 13:00
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.

2 participants