Skip to content
This repository was archived by the owner on Mar 17, 2026. It is now read-only.

fix: rename useProvenWithdrawal hook and correct Synapse image URL#1682

Open
MarsonKotovi4 wants to merge 6 commits intobase:masterfrom
MarsonKotovi4:master
Open

fix: rename useProvenWithdrawal hook and correct Synapse image URL#1682
MarsonKotovi4 wants to merge 6 commits intobase:masterfrom
MarsonKotovi4:master

Conversation

@MarsonKotovi4
Copy link
Copy Markdown

What changed? Why?

  • Renamed the export from useprovenWithdrawal to useProvenWithdrawal for consistency.
  • Updated all references in useWithdrawalStatus.ts to match the new name.
  • Corrected a typo in ecosystem.json by changing the image URL from /images/partners/synpase.webp to /images/partners/synapse.webp.

Notes to reviewers

  • Please verify the hook name now follows the project’s naming conventions.
  • Double-check that the updated image path is valid and displays correctly.

How has it been tested?

  • Ran a local build to ensure no compile errors.
  • Confirmed the hook usage in the withdrawal flow works as expected.
  • Verified the updated image path renders without errors.

@vercel
Copy link
Copy Markdown

vercel bot commented Jan 31, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
web-base-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 7, 2025 7:16pm

@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented Jan 31, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 2/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@vercel
Copy link
Copy Markdown

vercel bot commented Jan 31, 2025

@MarsonKotovi4 is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.

@MarsonKotovi4
Copy link
Copy Markdown
Author

@cb-heimdall

@MarsonKotovi4
Copy link
Copy Markdown
Author

@ryandotsmith

@wbnns wbnns added the base web review To be reviewed by the Base web team label Feb 5, 2025
@wbnns
Copy link
Copy Markdown
Contributor

wbnns commented Feb 5, 2025

@MarsonKotovi4 Thanks! The Synapse image was fixed in #1702 🙏

@MarsonKotovi4
Copy link
Copy Markdown
Author

@wbnns ok! fixed!

@dschlabach
Copy link
Copy Markdown
Contributor

Thanks!

@dschlabach dschlabach enabled auto-merge (squash) March 6, 2025 15:52
@brendan-defi
Copy link
Copy Markdown
Collaborator

@MarsonKotovi4 thanks for your submission. We'll happily merge this in once you resolve the lint issues and rebase to the latest master. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

base web review To be reviewed by the Base web team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants