Skip to content

feat: DEFI-1902: Add support for ICRC-10#178

Open
mbjorkqvist wants to merge 2 commits intomainfrom
mathias/DEFI-1902-add-icrc-10-support
Open

feat: DEFI-1902: Add support for ICRC-10#178
mbjorkqvist wants to merge 2 commits intomainfrom
mathias/DEFI-1902-add-icrc-10-support

Conversation

@mbjorkqvist
Copy link
Copy Markdown
Collaborator

Add support for the ICRC-10 standard, which basically consists of adding an icrc10_supported_standards query endpoint.

@mbjorkqvist mbjorkqvist marked this pull request as ready for review May 7, 2026 14:00
@mbjorkqvist mbjorkqvist requested a review from a team as a code owner May 7, 2026 14:00
Copy link
Copy Markdown
Member

@marc0olo marc0olo left a comment

Choose a reason for hiding this comment

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

ICRC-10 implementation looks correct: proper return type, self-listing of ICRC-10 in the standards list, both endpoints returning the same data via the shared helper, DID updated. Two minor pre-existing URL issues flagged inline.

Comment thread cycles-ledger/src/main.rs
},
endpoints::SupportedStandard {
name: "ICRC-103".to_string(),
url: "https://github.com/dfinity/ICRC/tree/main/ICRCs/ICRC-103".to_string(),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Pre-existing, but since you are touching this list: a tree/ URL resolves to a directory listing, not a document. The file exists at blob/main: https://github.com/dfinity/ICRC/blob/main/ICRCs/ICRC-103/ICRC-103.md

@marc0olo
Copy link
Copy Markdown
Member

marc0olo commented May 8, 2026

Pre-existing as well, but worth a follow-up: the ICRC-106 URL points to a closed (unmerged) GitHub PR. The spec file has since landed on main: https://github.com/dfinity/ICRC/blob/main/ICRCs/ICRC-106/ICRC-106.md

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