Skip to content

Feat: Add GoodBridge page to src/pages/gd #628#629

Open
Gutopro wants to merge 2 commits intoGoodDollar:masterfrom
Gutopro:master
Open

Feat: Add GoodBridge page to src/pages/gd #628#629
Gutopro wants to merge 2 commits intoGoodDollar:masterfrom
Gutopro:master

Conversation

@Gutopro
Copy link

@Gutopro Gutopro commented Mar 24, 2026

Description

This PR integrates the MessagePassingBridge

About #628

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

Checklist:

  • PR title matches follow: (Feature|Bug|Chore) Task Name
  • My code follows the style guidelines of this project
  • I have followed all the instructions described in the initial task (check Definitions of Done)
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • I have added reference to a related issue in the repository
  • I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
  • I have added screenshots related to my pull request (for frontend tasks)
  • I have pasted a gif showing the feature.
  • @mentions of the person or team responsible for reviewing proposed changes

image desktop view
image
mobile view

Summary by Sourcery

Add a new GoodBridge page that integrates the message-passing bridge into the GoodDollar app and expose it via navigation and routing.

New Features:

  • Introduce a GoodBridge page that renders the message-passing bridge experience within the app when a wallet is connected.
  • Add a Good Bridge navigation entry and route at /goodbridge to access the new bridge page.
  • Gate the GoodBridge functionality behind a new goodBridgeEnabled feature flag with per-network configuration.

Enhancements:

  • Update localized catalogs to include text used by the new GoodBridge page.

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey - I've left some high level feedback:

  • The new src/pages/gd/GoodBridge/index.tsx~ file looks like an editor backup/temporary file and should be removed from the PR.
  • On the GoodBridge page, the placeholder text says 'Connect a wallet to see your portfolio', which is confusing for a bridge view; consider updating the copy to mention bridging instead of portfolio.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- The new `src/pages/gd/GoodBridge/index.tsx~` file looks like an editor backup/temporary file and should be removed from the PR.
- On the GoodBridge page, the placeholder text says 'Connect a wallet to see your portfolio', which is confusing for a bridge view; consider updating the copy to mention bridging instead of portfolio.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

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.

1 participant