Skip to content

fix network name detection#344

Merged
peterjah merged 1 commit intomainfrom
fix_network_detection
Jan 16, 2026
Merged

fix network name detection#344
peterjah merged 1 commit intomainfrom
fix_network_detection

Conversation

@peterjah
Copy link
Contributor

No description provided.

Copy link
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.

Pull request overview

This PR fixes network name detection by standardizing the chainID/chainId field naming and updating the network type detection logic to use the @massalabs/massa-web3 library.

Changes:

  • Renamed chainID to chainId in TypeScript interface for consistency with JavaScript naming conventions
  • Refactored network type detection logic to use getNetworkNameByChainId from @massalabs/massa-web3
  • Added @massalabs/massa-web3 as a dependency

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
plugin/home/src/types/server.ts Changed field name from chainID to chainId in NetworkInfo interface
plugin/home/src/Status.tsx Refactored network detection logic to use massa-web3 library and updated to use chainId
plugin/home/src/App.tsx Updated chain ID field reference in network comparison logic
plugin/home/package.json Added @massalabs/massa-web3 dependency
plugin/home/package-lock.json Added package lock entries for new dependency and its transitive dependencies
Files not reviewed (1)
  • plugin/home/package-lock.json: Language not supported

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

@peterjah peterjah force-pushed the fix_network_detection branch from cc2502a to 67c85b1 Compare January 16, 2026 13:26
@peterjah peterjah merged commit 39585b7 into main Jan 16, 2026
15 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.

3 participants