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: use wallet #31

Merged
merged 11 commits into from
Sep 17, 2024
Merged

feat: use wallet #31

merged 11 commits into from
Sep 17, 2024

Conversation

LeslieOA
Copy link
Collaborator

@LeslieOA LeslieOA commented Sep 17, 2024

  • New icon
  • use-wallet integration
    • WalletConnect, Pera, Defly, Lute, Kibisis and Casa
  • Post new conversations (800 characters max)
  • Post new topics (60 characters for topic name)
  • Post replies to conversations and topics
  • User (handle) switching

Important

Currently untested on Lute and there's an issue with connecting Kibisis (previously communicated with the wallet creator)

Wallet Selection

Wallet Selection Wallet Selection

Handle Selection

Handle Selection

New Conversation (posting)

New Conversation (posting)

New Topic
New Topic

New Reply
New Reply

closes #35

- Set handle before posting
- Post transaction UI indicator
- Store remembers which handles are associated with which wallets, so: lo-fi user switching
- Fixed undefined values on `Replies.tsx` when switching between broadcast channels
- Fixed hardcoded `dateTime` metadata on `<time>`
@LeslieOA LeslieOA marked this pull request as ready for review September 17, 2024 07:22
@@ -4,7 +4,8 @@
<head>
<meta charset="utf-8" />
<link rel="icon" href="/favicon.svg" />
<meta name="viewport" content="width=device-width, initial-scale=1" />
<!-- <meta name="viewport" content="width=device-width, initial-scale=1" /> -->

Choose a reason for hiding this comment

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

should we remove this comment?

@HashMapsData2Value HashMapsData2Value merged commit 4e211c8 into main Sep 17, 2024
1 check passed
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.

use-wallet flow connection issues
2 participants