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

Receiver does not get memo for intraledger payments #2449

Open
Extheoisah opened this issue Mar 16, 2023 · 4 comments
Open

Receiver does not get memo for intraledger payments #2449

Extheoisah opened this issue Mar 16, 2023 · 4 comments
Labels
enhancement New feature or request

Comments

@Extheoisah
Copy link

When I receive an intraledger payment, the memo field in the transactions data that is returned from the backend is null even when the sender sets a memo/note when making the payment. The sender has the memo in his/her transaction details but not the receiver.

We might want to also show the note that was set by the sender, so we need to send the memo field as part of the transaction data. The frontend already expects this data here https://github.com/GaloyMoney/galoy-mobile/blob/ace5b5de465720f256b4730d5001ed6c8cdca747/app/components/transaction-item/transaction-item.tsx#L76

@Extheoisah Extheoisah added the enhancement New feature or request label Mar 16, 2023
@Marvel-Ib
Copy link

Hi @Extheoisah I would love to take up the issue .

just to clarify the receiver does not get the memo set by the sender at the moment and the aim is to ensure whenever a sender sets a memo/note the receiver is able to see the memo/note as well.

@nicolasburtey
Copy link
Member

hi @Marvel-Ib. thanks for your message and wanting to contribute to our repo. it's probably a complex issue. there are privacy implication we need to narrow down here.

I think @vindard may have more context around the memo issue, but if you want to tackle some issues to get started with our repo, there might be easier one

@vindard
Copy link
Contributor

vindard commented Mar 16, 2023

Agree with @nicolasburtey, there might be easier issues to tackle. If we want to do this properly we'd likely need multiple "memo" fields on a single entry, some of which would be walletId-private, e.g.:

  • privateMemo
  • sharedMemo
  • memoFromLnInvoice

@openoms
Copy link
Collaborator

openoms commented Dec 6, 2023

Despite adding a note to the invoice (which correctly shows up on the sender side) the memo field is not being stored on the receiver side neither for intraLedger nor for ligthning transactions.
Also noted in the review here: getAlby/lightning-browser-extension#2887 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants