Skip to content

fix(statics): process every token per family in createTokenMapUsingTr… - #9738

Merged
hrishikeshjain merged 1 commit into
masterfrom
hrishikeshjain/chalo-1537-statics-token-map-family-drop
Sep 15, 2026
Merged

hrishikeshjain merged 1 commit into
masterfrom
hrishikeshjain/chalo-1537-statics-token-map-family-drop

Conversation

@hrishikeshjain

Copy link
Copy Markdown
Contributor

…immedConfigDetails

Only tokenConfigs[0] was read per family, so AMS tokens sharing a family (e.g. multiple Canton tokens) beyond the first were silently dropped from the resulting CoinMap with no error or log.

Ticket: CHALO-1537

@hrishikeshjain
hrishikeshjain requested a review from a team as a code owner September 15, 2026 11:52
@linear-code

linear-code Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

CHALO-1537

…immedConfigDetails

Only tokenConfigs[0] was read per family, so AMS tokens sharing a
family (e.g. multiple Canton tokens) beyond the first were silently
dropped from the resulting CoinMap with no error or log.

Add a regression test covering multiple tokens grouped under one
family key.

Ticket: CHALO-1537
@hrishikeshjain
hrishikeshjain force-pushed the hrishikeshjain/chalo-1537-statics-token-map-family-drop branch from 85f6b0b to af82cdb Compare September 15, 2026 11:57
@github-actions

github-actions Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

⚠️ Unit tests are failing on Node 26.x (Current release line, non-blocking). This is not an LTS version yet, so it does not block merge, but it signals an incompatibility to fix before Node 26.x becomes LTS.

View run

@hrishikeshjain
hrishikeshjain merged commit 00ff451 into master Sep 15, 2026
26 checks passed
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