Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: added bebop dex #1728

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KirillTsonev
Copy link

What changed? Why?
Added bebop.xyz to ecosystem
Notes to reviewers

How has it been tested?
Locally

@cb-heimdall
Copy link
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
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 1
Sum 2

Copy link

vercel bot commented Feb 7, 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 20, 2025 7:37pm

Copy link

vercel bot commented Feb 7, 2025

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

A member of the Team first needs to authorize it.

@wbnns
Copy link
Contributor

wbnns commented Mar 5, 2025

@KirillTsonev

Hi there! Please add a Base logo to the landing page to make it clear that the network is supported, for users that would be arriving from base.org.

@wbnns wbnns added the change(s) requested One or more changes have been requested label Mar 5, 2025
@KirillTsonev
Copy link
Author

@wbnns Hello! Can you please elaborate a bit? We have Base listed on our homepage in the second section
image
and following the link provided in the application will automatically set the network to Base as well
image
Thank you.

@wbnns
Copy link
Contributor

wbnns commented Mar 25, 2025

@KirillTsonev

Thanks for following up! Apologies, it isn't clearly visible on the landing page, above the fold, where Base users would be getting directed. Would it be possible to update the landing page so that the user arrives on the trading interface with Base already selected?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
base ecosystem review This is ecosystem related change(s) requested One or more changes have been requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants