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

dapp listing: add sarafu network #467

Merged
merged 5 commits into from
Aug 29, 2023
Merged

Conversation

kamikazechaser
Copy link
Contributor

@kamikazechaser kamikazechaser commented Jul 3, 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 V1 or DAppKit;
  • must be mobile-first or mobile responsive;
  • is in one of the existing categories; and
  • includes a link to user documentation.

@kamikazechaser kamikazechaser marked this pull request as ready for review July 4, 2023 02:34
@MuckT MuckT added the status: in review The Valora team is reviewing your dapp label Jul 15, 2023
@MuckT
Copy link
Collaborator

MuckT commented Jul 15, 2023

@kamikazechaser on sarafu.network the 'Read More' option goes to a technical document. Would it be possible to replace this with something less technical such as the blog?

@kamikazechaser
Copy link
Contributor Author

Sure, we will get that updated.

@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 Aug 3, 2023
@MuckT
Copy link
Collaborator

MuckT commented Aug 3, 2023

@kamikazechaser is this still being worked on?

@kamikazechaser
Copy link
Contributor Author

Updated "Read More" -> https://docs.grassecon.org/

@kamikazechaser
Copy link
Contributor Author

kamikazechaser commented Aug 24, 2023

@MuckT Let me know if we need to change anything else. I'll fix the conflicts.

@MuckT MuckT added status: in review The Valora team is reviewing your dapp and removed status: address feedback The Valora team left feedback to address labels Aug 28, 2023
@MuckT
Copy link
Collaborator

MuckT commented Aug 28, 2023

@kamikazechaser looks like a small formatting issue please run yarn format and commit the changes.

@kamikazechaser
Copy link
Contributor Author

@MuckT Done!

@MuckT MuckT added status: accepted The Valora team has fully accepted your dapp and removed status: in review The Valora team is reviewing your dapp labels Aug 29, 2023
@MuckT
Copy link
Collaborator

MuckT commented Aug 29, 2023

@kamikazechaser thanks for all your hard work. Glad to see this getting merged!

@MuckT MuckT merged commit 15cc3ea into valora-inc:main Aug 29, 2023
@kamikazechaser
Copy link
Contributor Author

Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: accepted The Valora team has fully accepted your dapp
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants