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

[Import Multisig] Added import multisig data logic #36

Merged
merged 3 commits into from
Aug 30, 2023

Conversation

matextrem
Copy link
Contributor

This PR has the following changes:

  • Added logic to query squid graphql API for getting multisig data.
  • Modify useFetchSignersAccount hook to query graphql multisigs data.
  • Modify handleOwners method to check for duplicated addresses.

@matextrem matextrem added the enhancement New feature or request label Aug 29, 2023
@matextrem matextrem self-assigned this Aug 29, 2023
@vercel
Copy link

vercel bot commented Aug 29, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ink-multisig-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 29, 2023 10:23pm
story-ink-multisig-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 29, 2023 10:23pm

@@ -16,6 +17,11 @@ import { PolkadotContextProvider } from "@/context/usePolkadotContext";
import ThemeCustomization from "@/themes";
import createEmotionCache from "@/utils/createEmotionCache";

export const squidClient = new ApolloClient({
Copy link
Collaborator

@henrypalacios henrypalacios Aug 29, 2023

Choose a reason for hiding this comment

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

@0xLucca We need to define the URL's by each network for squid.

TODO

Copy link
Collaborator

Choose a reason for hiding this comment

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

For now we only have the Shibuya endpoint and its not final. Once we have a release we will deploy the squids to multiple networks.

Copy link
Collaborator

@henrypalacios henrypalacios left a comment

Choose a reason for hiding this comment

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

It is almost ready to go,
I only ask you to once the wallet has been raised, to press confirm before sending to the App dashboard set this wallet as selected with setXsigner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants