Skip to content

Update README.md#394

Merged
Dargon789 merged 2 commits intomasterfrom
Dargon789-patch-1
Mar 19, 2026
Merged

Update README.md#394
Dargon789 merged 2 commits intomasterfrom
Dargon789-patch-1

Conversation

@Dargon789
Copy link
Copy Markdown
Owner

@Dargon789 Dargon789 commented Mar 19, 2026

Dargon789/demo-dapp@0e54e91

Summary by Sourcery

Revise the embedded README for the demo dapp to incorporate Wagmi project information, community and contribution links, update references to the demo app and source file, and remove the legacy license section.

Documentation:

  • Expand the README with Wagmi branding, documentation, community, contributing, and sponsorship information.
  • Clarify the description and link for the Sequence Wallet demo dapp and update the source reference to the GitHub App.tsx path.

Dargon789/demo-dapp@0e54e91

Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
@bolt-new-by-stackblitz
Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@codesandbox
Copy link
Copy Markdown

codesandbox bot commented Mar 19, 2026

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@gemini-code-assist
Copy link
Copy Markdown

Warning

Gemini encountered an error creating the summary. You can try again by commenting /gemini summary.

@vercel

This comment was marked as resolved.

@sourcery-ai
Copy link
Copy Markdown

sourcery-ai bot commented Mar 19, 2026

Reviewer's Guide

Updates the demo dapp README to reuse Wagmi’s branded README content, expand documentation/community/contributing sections, adjust links to external resources, and remove the explicit license section.

Flow diagram for navigation paths introduced in the updated README

flowchart TD
  A[README landing section<br/>- Wagmi branding<br/>- Badges] --> B[Documentation<br/>wagmi.sh]
  A --> C[Demo description<br/>Sequence Wallet usage]

  B --> D[GitHub Discussions<br/>wevm/wagmi/discussions]
  B --> E[Wagmi contributing guide<br/>wagmi.sh/dev/contributing]

  C --> F[Hosted demo dapp<br/>0xsequence.github.io/demo-dapp]
  C --> G[App source code<br/>src/App.tsx on GitHub]

  A --> H[Community links]
  H --> I[Wagmi Discord]

  A --> J[Sponsorship<br/>GitHub Sponsors for Wagmi]
Loading

File-Level Changes

Change Details Files
Replace the minimal demo-dapp description with Wagmi-branded README header, badges, and project marketing copy while retaining a short Sequence demo description at the bottom.
  • Add HTML comment callout about Wagmi’s Gitcoin Grants participation.
  • Insert centered Wagmi logo, tagline, and multiple status badges (version, license, downloads, Best of JS, Codecov).
  • Add sections for Documentation, Community, Contributing, and Sponsors with links into the Wagmi ecosystem.
  • Reintroduce and slightly tweak the original Sequence demo-dapp description and link after the Wagmi content.
.codesandbox/src/README.md
Adjust project-specific links and remove local LICENSE section.
  • Change the reference to App.tsx from a relative path to an absolute GitHub URL for the demo-dapp repository.
  • Remove the explicit LICENSE section that previously stated Apache 2.0 or MIT.
  • Apply minor spacing/whitespace tweaks around screenshots and headings.
.codesandbox/src/README.md

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sequence-js-docs Ready Ready Preview, Comment Mar 19, 2026 10:15pm
sequence-js-web Ready Ready Preview, Comment Mar 19, 2026 10:15pm
sequence.js Ready Ready Preview, Comment Mar 19, 2026 10:15pm
wagmi-project Error Error Mar 19, 2026 10:15pm

@snyk-io
Copy link
Copy Markdown

snyk-io bot commented Mar 19, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Copy link
Copy Markdown

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey - I've found 1 issue, and left some high level feedback:

  • There is an orphaned closing </a> tag before the Sequence sponsor block that should be removed or paired with a corresponding opening tag to avoid malformed HTML in the rendered README.
  • The README now heavily foregrounds Wagmi branding and project metadata at the top, which may be confusing in the context of a Sequence Wallet demo dapp; consider adding a short clarifying section or heading that explicitly distinguishes the demo’s purpose from the Wagmi project information.
  • The commented Gitcoin Grants round 21 notice at the top is time-bound and likely to become stale; consider either removing it or rephrasing to a more evergreen support link without a specific round reference.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- There is an orphaned closing `</a>` tag before the Sequence sponsor block that should be removed or paired with a corresponding opening tag to avoid malformed HTML in the rendered README.
- The README now heavily foregrounds Wagmi branding and project metadata at the top, which may be confusing in the context of a Sequence Wallet demo dapp; consider adding a short clarifying section or heading that explicitly distinguishes the demo’s purpose from the Wagmi project information.
- The commented Gitcoin Grants round 21 notice at the top is time-bound and likely to become stale; consider either removing it or rephrasing to a more evergreen support link without a specific round reference.

## Individual Comments

### Comment 1
<location path=".codesandbox/src/README.md" line_range="88" />
<code_context>
+</p>
+
+
+Demo app example on how to use Sequence Wallet. Covers how to connect, sign messages and send transactions.
+
+Try this dapp at: [https://0xsequence.github.io/demo-dapp](https://0xsequence.github.io/demo-dapp)
</code_context>
<issue_to_address>
**suggestion (typo):** The phrase "Demo app example on how to use" is slightly awkward grammatically.

Consider rephrasing to something like "Demo app showing how to use Sequence Wallet" or "Demo app example of how to use Sequence Wallet" for smoother grammar and readability.

```suggestion
Demo app showing how to use Sequence Wallet. It covers how to connect, sign messages, and send transactions.
```
</issue_to_address>

Fix all in Cursor


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
@Dargon789 Dargon789 merged commit fc1b3d9 into master Mar 19, 2026
13 of 17 checks passed
@Dargon789 Dargon789 deleted the Dargon789-patch-1 branch March 19, 2026 16:38
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.

1 participant