Skip to content

Conversation

@fmorency
Copy link
Contributor

This pull request updates several dependencies and makes minor adjustments to the UI and authentication configuration. The most significant changes are dependency upgrades for wallet and client packages, along with a small UI color tweak and a configuration fix for Web3Auth.

Dependency updates:

  • Updated @cosmos-kit/web3auth to use a custom package version from the @manifest-network namespace (2.16.6-ll.1) in package.json.
  • Upgraded @skip-go/client to 1.5.9 and @skip-go/widget to 3.14.14 in package.json.

UI and configuration improvements:

  • Added a background color (#161616) to the success style in IbcSendForm for improved visual consistency.
  • Moved the mfaLevel: 'optional' property to the correct location within the client configuration object in Web3AuthProvider for accurate multi-factor authentication setup.

@fmorency fmorency requested a review from Copilot October 17, 2025 17:18
@fmorency fmorency self-assigned this Oct 17, 2025
@vercel
Copy link

vercel bot commented Oct 17, 2025

@fmorency is attempting to deploy a commit to the Manifest Network's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates wallet and IBC widget dependencies, fixes a Web3Auth configuration issue, and adds a background color for consistency in the IBC send form.

Key changes:

  • Upgraded @skip-go/client from 1.3.7 to 1.5.9 and @skip-go/widget from 3.12.11 to 3.14.14
  • Switched @cosmos-kit/web3auth to a custom package from @manifest-network namespace
  • Corrected placement of mfaLevel property in Web3Auth configuration

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
package.json Updates dependencies for Web3Auth and Skip-Go packages
contexts/web3AuthContext.tsx Moves mfaLevel property into the correct client configuration object
components/bank/forms/ibcSendForm.tsx Adds background color to success state styling

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@codecov
Copy link

codecov bot commented Oct 17, 2025

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
components/bank/forms/ibcSendForm.tsx 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@fmorency fmorency requested review from chalabi2 and Copilot October 17, 2025 17:41
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 5 out of 6 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@fmorency fmorency merged commit 467261f into manifest-network:main Oct 20, 2025
4 of 6 checks passed
@fmorency fmorency deleted the web3auth-v10 branch October 20, 2025 13:47
mergify bot pushed a commit that referenced this pull request Oct 20, 2025
mergify bot added a commit that referenced this pull request Oct 20, 2025
(cherry picked from commit 467261f)

Co-authored-by: Félix C. Morency <1102868+fmorency@users.noreply.github.com>
mergify bot pushed a commit that referenced this pull request Oct 23, 2025
mergify bot added a commit that referenced this pull request Oct 23, 2025
(cherry picked from commit 467261f)

Co-authored-by: Félix C. Morency <1102868+fmorency@users.noreply.github.com>
mergify bot pushed a commit that referenced this pull request Oct 27, 2025
mergify bot added a commit that referenced this pull request Oct 27, 2025
(cherry picked from commit 467261f)

Co-authored-by: Félix C. Morency <1102868+fmorency@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant