Skip to content

feat(vara-eth-idea/frontend): add message page#2297

Merged
nikitayutanov merged 16 commits intomainfrom
ny-message
Jan 23, 2026
Merged

feat(vara-eth-idea/frontend): add message page#2297
nikitayutanov merged 16 commits intomainfrom
ny-message

Conversation

@nikitayutanov
Copy link
Member

No description provided.

@nikitayutanov nikitayutanov added C1-feature Feature request D6-vara-eth-idea Vara.Eth Idea labels Jan 15, 2026
@nikitayutanov
Copy link
Member Author

New ChainEntity space contains some components with logic that already exists at other parts of the app, such as Code or Program pages.

Those parts are gonna be refactored to reuse ChainEntity components and to drop repetitive logic right in the next PR.

Same thing applies to PageContainer component. I'm not quite sure how to compose it yet for comfortable reuse across the app, but will take a closer look after above mentioned refactoring I guess.

Copy link

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 adds a new message detail page to the vara-eth frontend application, allowing users to view comprehensive information about messages and replies. The changes include significant refactoring of message-related API queries, introduction of reusable UI components for chain entities, and routing updates.

Changes:

  • Created a new /messages/:messageId route with a dedicated message detail page
  • Refactored message queries to separate API requests from React Query hooks and added support for replies
  • Introduced reusable UI components (PageContainer, ChainEntity, ExplorerLink) for displaying blockchain entity information

Reviewed changes

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

Show a summary per file
File Description
vite-env.d.ts Removed type reference file (likely consolidated elsewhere)
shared/utils/index.ts Added export for clsx utility as cx
shared/types/index.ts Added PropsWithClassName type definition
shared/config/routes.ts Added message route path
react-router.d.ts Added type augmentation for react-router's NavigateFunction
pages/user/user.tsx Updated type casting for userId parameter
pages/message/* Created new message page with arrow icon and styling
pages/index.tsx Added message route to routing configuration
pages/home/home.tsx Updated to use refactored query hooks and include reply counts
features/messages/lib/requests.ts Created new file with API request functions and type definitions
features/messages/lib/queries.ts Refactored to separate concerns and added reply-related queries
features/messages/lib/index.ts Updated exports to include new types and hooks
features/messages/index.ts Added MessageData component export
features/messages/components/message-data/* Created component to render different message types
components/ui/page-container/* Created reusable page container component
components/ui/chain-entity/* Created reusable components for displaying chain entity information
components/ui/explorer-link/* Created component for Etherscan links
components/ui/hash-link/* Enhanced to support explorer links
components/ui/button/index.ts Exported ButtonProps type
components/header/* Made logo clickable with link to home
app/app.tsx Added page class to layout div
app/app.css Added flex layout styles for page container

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

@nikitayutanov nikitayutanov changed the base branch from main to ny-search January 16, 2026 14:16
Base automatically changed from ny-search to main January 19, 2026 12:23
@nikitayutanov nikitayutanov merged commit 97cbfc0 into main Jan 23, 2026
23 of 24 checks passed
@nikitayutanov nikitayutanov deleted the ny-message branch January 23, 2026 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C1-feature Feature request D6-vara-eth-idea Vara.Eth Idea

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants