Skip to content

fix: change symbol from goqxmp to qxmp#8816

Merged
damodarnaik699 merged 1 commit into
masterfrom
SCAAS-9437
May 20, 2026
Merged

fix: change symbol from goqxmp to qxmp#8816
damodarnaik699 merged 1 commit into
masterfrom
SCAAS-9437

Conversation

@damodarnaik699

@damodarnaik699 damodarnaik699 commented May 20, 2026

Copy link
Copy Markdown
Contributor

@damodarnaik699 damodarnaik699 requested a review from a team as a code owner May 20, 2026 09:37
@linear-code

linear-code Bot commented May 20, 2026

Copy link
Copy Markdown

SCAAS-9437

@damodarnaik699

Copy link
Copy Markdown
Contributor Author

@claude review this pr

@damodarnaik699

Copy link
Copy Markdown
Contributor Author

@copilot review this pr throughly

@vipulchaursiya908 vipulchaursiya908 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.

LGTM

@vipulchaursiya908

Copy link
Copy Markdown

@claude can you give approval rating

Copilot AI commented May 20, 2026

Copy link
Copy Markdown
Contributor

@copilot review this pr throughly

Completed a thorough review on commit e312bc7. I verified symbol/underlying-asset/address updates are consistent across base.ts, erc20Coins.ts, ofcErc20Coins.ts, and naming-convention tests; no goqxmp references remain. Validation run: modules/statics targeted unit test (tokenNamingConvention.ts) passed, and modules/statics build passed. No additional code changes are required.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • download.cypress.io
    • Triggering command: /opt/hostedtoolcache/node/24.14.1/x64/bin/node /opt/hostedtoolcache/node/24.14.1/x64/bin/node index.js --exec install (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@damodarnaik699 damodarnaik699 merged commit 6add7f1 into master May 20, 2026
23 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.

5 participants