Skip to content

Conversation

fleandrei
Copy link
Contributor

No description provided.

@fleandrei fleandrei requested a review from peterjah September 18, 2025 16:20
@fleandrei fleandrei self-assigned this Sep 18, 2025
@peterjah peterjah requested a review from Copilot September 18, 2025 20:05
Copy link

@Copilot 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 introduces a new useResolveDeweb React hook that resolves DeWeb URLs using the massa-web3 library. The hook provides functionality to convert massa.network domain URLs to their resolved counterparts with proper loading states and error handling.

  • Adds useResolveDeweb hook with URL resolution capabilities and state management
  • Creates a extractMNSUrl utility function to parse massa.network URLs
  • Refactors the export structure to use a centralized hooks index file

Reviewed Changes

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

Show a summary per file
File Description
src/lib/massa-react/index.ts Consolidated exports to use hooks index file
src/lib/massa-react/hooks/useResolveDeweb.ts New hook implementation with URL resolution logic
src/lib/massa-react/hooks/useResolveDeweb.spec.ts Comprehensive test suite for the extractMNSUrl function
src/lib/massa-react/hooks/index.ts Centralized exports for all hooks
.eslintrc.cjs Disabled indent rule

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

@Copilot 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

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


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@fleandrei fleandrei requested a review from Copilot September 22, 2025 09:37
Copy link

@Copilot 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

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


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@fleandrei fleandrei requested a review from Copilot September 22, 2025 10:56
Copy link

@Copilot 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

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


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@fleandrei fleandrei requested a review from Copilot September 22, 2025 11:04
Copy link

@Copilot 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

Copilot reviewed 5 out of 6 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@fleandrei fleandrei merged commit d85c2c8 into main Sep 22, 2025
4 checks passed
@fleandrei fleandrei deleted the add_useResolveDeweb_hook branch September 22, 2025 11:58
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