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

Python/JS code sample for Memo #1685

Merged
merged 15 commits into from
Feb 7, 2023
Merged

Python/JS code sample for Memo #1685

merged 15 commits into from
Feb 7, 2023

Conversation

wojake
Copy link
Contributor

@wojake wojake commented Jan 20, 2023

Validate and send a Memo.

This is work for a XRPL Bounty @ XRPLBounties/Proposals for Milestone 7 and XRPLBounties/Proposals for Milestone 10

@mDuo13 mDuo13 added the bounties Issues or PRs relating to the XRPL Bounties program, including PRs intended to claim a bounty label Jan 20, 2023
@wojake wojake changed the title Python code sample for Memo Python/JS code sample for Memo Jan 20, 2023
small changes in order for the js code sample to be comparable to the python code sample
@JST5000
Copy link
Contributor

JST5000 commented Jan 30, 2023

Not sure if there's a good spot to put this information, but Memos are only viewable if you ask a node that "remembers" the ledger where the transaction was submitted. So, full-history nodes will always remember, and for recent transactions most validators will be able to look up memos with the tx command, but they don't get stored in the blockchain ledger itself the way LedgerObjects are for example. (So, you may have to query full history nodes if you want to do something like search for transactions with a certain memo)

@jonathanlei jonathanlei merged commit 627b5e2 into XRPLF:master Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bounties Issues or PRs relating to the XRPL Bounties program, including PRs intended to claim a bounty
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants