Skip to content

docs: add ledger index canister IDs to deployment table#173

Open
lpahlavi wants to merge 1 commit intomainfrom
lpahlavi/readme-index-canister-ids
Open

docs: add ledger index canister IDs to deployment table#173
lpahlavi wants to merge 1 commit intomainfrom
lpahlavi/readme-index-canister-ids

Conversation

@lpahlavi
Copy link
Copy Markdown
Contributor

Summary

  • Adds the cksol_index canister ID to both the staging and production deployment tables

IDs sourced from .icp/data/mappings/{staging,prod}.ids.json.

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 24, 2026 08:59
@lpahlavi lpahlavi requested review from THLO and maciejdfinity April 24, 2026 09:03
@lpahlavi lpahlavi marked this pull request as ready for review April 24, 2026 09:03
@lpahlavi lpahlavi requested a review from a team as a code owner April 24, 2026 09:03
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Comment thread README.md
|----------|-------------|
| Minter | `lh22c-kyaaa-aaaar-qb5nq-cai` *(not yet deployed)* |
| Ledger | `ls5lp-lqaaa-aaaar-qb5oa-cai` *(not yet deployed)* |
| Ledger Index | `2ezyf-hqaaa-aaaar-qb6ga-cai` *(not yet deployed)* |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I'm not sure what the *(not yet deployed)* means currently - the canisters identified by Ledger and Ledger Index in this table do seem to be running ledger and index wasms, respectively, but perhaps they still need to be redeployed for production (and be put under NNS control) before removing that comment. Also, the ledger should be upgraded to include the index canister ID, so that calls to its icrc106_get_index_principal endpoint returns the index' canister ID (currently it returns an IndexPrincipalNotSet error).

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.

3 participants