Skip to content

Conversation

@egemenkus
Copy link
Contributor

Previously, when clicking on one of the accounts listed under the /settings/manager page, the app was generating a malformed URL and returning a 400 error.

❌ Broken URL example:

/settings/manager/[object Object]/address/0x4A1a2197f307222cD67A1762D9A352F64558d9Be

The anchor tag being rendered looked like this:

<a
  rel="noreferrer noopener"
  href="/settings/manager/[object Object]/address/0x4A1a2197f307222cD67A1762D9A352F64558d9Be"
  target="_blank"
  data-discover="true"
/>

✅ Fix applied

After updating the WalletAccount.tsx component, the URL is now correctly generated using the block explorer's base URL and the manager address.

✅ Corrected anchor output:

<a
  rel="noreferrer noopener"
  href="https://explorer.testnet.lens.xyz/address/0x4A1a2197f307222cD67A1762D9A352F64558d9Be"
  target="_blank"
/>

@bigint bigint requested a review from Copilot April 14, 2025 07:58
@bigint bigint changed the title fix(wallet): correct block explorer URL in WalletAccount component fix: correct block explorer URL in WalletAccount component Apr 14, 2025
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.

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

@bigint
Copy link
Owner

bigint commented Apr 14, 2025

Epic thanks @egemenkus 🙏🏼

@bigint bigint merged commit 6676844 into bigint:main Apr 14, 2025
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