Skip to content

feat(statics): add SoFiUSD Canton token (staging, test, prod)#9124

Open
zeeshanamjad-eng wants to merge 1 commit into
masterfrom
feat/SCAAS-9964-onboard-sofiusd
Open

feat(statics): add SoFiUSD Canton token (staging, test, prod)#9124
zeeshanamjad-eng wants to merge 1 commit into
masterfrom
feat/SCAAS-9964-onboard-sofiusd

Conversation

@zeeshanamjad-eng

Copy link
Copy Markdown
Contributor

Ticket: SCAAS-9964

@zeeshanamjad-eng zeeshanamjad-eng requested a review from a team as a code owner June 26, 2026 13:37
@linear-code

linear-code Bot commented Jun 26, 2026

Copy link
Copy Markdown

SCAAS-9964

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds the SoFiUSD token to the statics registry for the Canton network and its OFC wrappers, covering production plus tcanton test/staging variants via distinct underlying assets and registrar endpoints.

Changes:

  • Registered canton:sofid (SoFiUSD) in cantonTokens with stablecoin features and its registrar reference.
  • Registered tcanton:sofid and tcanton:stgsofid test tokens with appropriate registrar base URLs and stablecoin features.
  • Added corresponding OFC coin entries (ofccanton:sofid, ofctcanton:sofid, ofctcanton:stgsofid) and extended UnderlyingAsset for the new symbols.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
modules/statics/src/coins/ofcCoins.ts Adds OFC coin definitions for SoFiUSD on Canton (prod) and tcanton (test/staging).
modules/statics/src/coins/cantonTokens.ts Adds Canton mainnet and tcanton test/staging token definitions for SoFiUSD, including registrar references and stablecoin features.
modules/statics/src/base.ts Extends UnderlyingAsset enum with canton:sofid, tcanton:sofid, and tcanton:stgsofid.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zeeshanamjad-eng

Copy link
Copy Markdown
Contributor Author

@claude

@github-actions

github-actions Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Claude encountered an error —— View job


Code Review for SoFiUSD Canton Token Addition

Todo List:

  • Read and analyze all changed files
  • Review token definitions for correctness and consistency
  • Check adherence to BitGoJS patterns and conventions
  • Provide comprehensive code review feedback

Analyzing the PR changes...

@zeeshanamjad-eng zeeshanamjad-eng force-pushed the feat/SCAAS-9964-onboard-sofiusd branch 2 times, most recently from 60e6f38 to ef22892 Compare June 26, 2026 13:47
Comment thread modules/statics/src/coins/cantonTokens.ts Outdated
damodarnaik699
damodarnaik699 previously approved these changes Jun 26, 2026

@damodarnaik699 damodarnaik699 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM -> Can you address that 1 minor comment

To keep consistence of sofid token with respect to other chains

Ticket: SCAAS-9964

Co-authored-by: damodarnaik699 <108988642+damodarnaik699@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comment thread modules/statics/src/coins/cantonTokens.ts
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