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.

Sign up for GitHub

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

Jump to bottom

Add Support for CTIDs (Concise Transaction Identifiers) #719

Open
ckniffen opened this issue Jun 27, 2023 · 5 comments
Open

Add Support for CTIDs (Concise Transaction Identifiers) #719

ckniffen opened this issue Jun 27, 2023 · 5 comments
Assignees
@cindyyan317
Labels
enhancement New feature or request
Milestone
Future

Comments

@ckniffen
Copy link

ckniffen commented Jun 27, 2023 •
edited
Loading

There is work being done to add a new way to look up a transaction. It uses the ledger, index on the ledger, and network id to look up a transaction.

XRPLF/rippled#4418
@ckniffen ckniffen changed the title Support for CTID (Concise Transaction Identifiers) Add Support for CTIDs (Concise Transaction Identifiers) Jun 27, 2023
@intelliot intelliot added this to the 2.1 milestone Aug 22, 2023
@intelliot intelliot added enhancement New feature or request v2.1 labels Aug 22, 2023
@godexsoft godexsoft removed the v2.1 label Aug 22, 2023
@cindyyan317
Copy link
Collaborator

tx handler needs to be changed.

@cindyyan317 cindyyan317 self-assigned this Sep 21, 2023
@cindyyan317
Copy link
Collaborator

cindyyan317 commented Oct 3, 2023 •
edited
Loading

ctid updated on rippled XRPLF/rippled#4738

Since XRPLF/rippled#4418 has been in 1.12, 4738 is still working. Clio will match 4418 output.

@intelliot
Copy link
Collaborator

The best developer experience would be to always return the ctid with validated transactions. (In Clio, that should be all transactions.)

@cindyyan317
Copy link
Collaborator

ctid support for tx added in #917

@godexsoft
Copy link
Collaborator

Whether this makes into 2.1 will depend on rippled implement the same logic first. CTID support in tx is already added.

@cindyyan317 cindyyan317 removed this from the 2.1 milestone Jan 8, 2024
@kuznetsss kuznetsss added this to the Future milestone Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cindyyan317 cindyyan317

Labels
enhancement New feature or request
Projects
Clio
Status: 📋 Backlog
Milestone
Future
Development

No branches or pull requests
5 participants
@intelliot @godexsoft @ckniffen @kuznetsss @cindyyan317