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: Initial integration docs #140

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

feat: Initial integration docs #140

wants to merge 1 commit into from

Conversation

gbarkhatov
Copy link
Contributor

Initial integration docs

Closes #16

@@ -5,6 +5,10 @@
</p>
<br/>

## Overview

The Babylon Wallet Connector provides a unified interface for integrating both Bitcoin (BTC) and Babylon (BBN) wallets into Babylon dApp. It supports both native wallet extensions and injectable mobile wallets.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"native wallet extensions" & "injectable mobile wallets" might be confusing for people outside of the company.
How about have a small section dedicate to explain what this really mean? and especially for the mobile injectable, give a small example with screenshot


### Integration Through Tomo Connect SDK Lite

In addition to direct integration, wallets can also be integrated through [Tomo Connect SDK Lite](https://docs.tomo.inc/tomo-sdk/tomo-connect-sdk-lite).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO we shall put this section first as we would like to delegate this to tomo as much as possible. i.e encourage wallet to go through tomo. if it doesn't work out due to xyz reason, come to us


This guide explains how to integrate wallets with the Babylon staking dApp. The dApp supports both Bitcoin (BTC) and Babylon (BBN) wallets through a unified interface.

### Supported Wallet Types
Copy link
Collaborator

@jrwbabylonlab jrwbabylonlab Dec 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's not put everything under a single README.
We can start with the high-level file structure first then gradually define the content.
Something on top of my head:

  • README.md: table of content of below points from "doc" directly. The rest can just be local dev instruction on how to spin up playbook, how to run tests etc.
  • doc
    1. bitcoin: instruction on how to get listed as part of the bitcoin wallets
    2. babylon: instruction on how to get listed as part of the BBN wallets
    3. mobile: instruction on how to work with mobile injectable where both BTC & BBN need to be implemented
    4. General guideline: This page highlights the rules wallets should follow before and after integration. For example, no breaking change, provide POC for incident etc. We can also put this as first line of the main README.md
    5. FAQs: A list of common questions that were asked by wallets

Note: no need to go detail into each section at this stage. I'm just highlighting we shall start with skeleton on the structure and areas we want to cover. The actual content can be added later on.

thoughts on the structure? @totraev @jeremy-babylonlabs @vitsalis

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.

Docs: Update wallet docs
3 participants