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

add Ethscription Forest to the dapp list #533

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ludomain
Copy link

@ludomain ludomain commented Sep 13, 2023

To make a submission to the Dapps Page, please complete the task lists
below; otherwise remove them.

Your Dapp:

  • connects to Valora via WalletConnect V2;
  • must be mobile-first or mobile responsive;
  • is in one of the existing categories; and
  • includes a link to user documentation.

For more information see the Valora Listing Overview.

@MuckT MuckT added the status: in review The Valora team is reviewing your dapp label Sep 13, 2023
@MuckT
Copy link
Collaborator

MuckT commented Sep 13, 2023

@ludomain it looks like your app is missing the following:

  • connects to Valora via WalletConnect V2 (Only MetaMask currently)
  • must be mobile-first or mobile responsive (Connect button is too small)
  • includes a link to user documentation (No link present on homepage)

@MuckT MuckT added status: address feedback The Valora team left feedback to address and removed status: in review The Valora team is reviewing your dapp labels Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: address feedback The Valora team left feedback to address
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants