Skip to content

Commit

Permalink
make skip hook more type safe
Browse files Browse the repository at this point in the history
  • Loading branch information
chalabi2 committed Feb 4, 2025
1 parent 3e39891 commit 625ce3a
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 5 deletions.
3 changes: 2 additions & 1 deletion components/bank/forms/ibcSendForm.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -109,9 +109,10 @@ export default function IbcSendForm({
const [isIconRotated, setIsIconRotated] = useState(false);

const getCosmosSigner = async () => {
const signer = chains[selectedFromChain.name].getOfflineSignerAmino();
const signer = chains[selectedFromChain.id].getOfflineSignerAmino();
return signer;
};

const skipClient = useSkipClient({
getCosmosSigner: getCosmosSigner,
});
Expand Down
12 changes: 8 additions & 4 deletions contexts/skipGoContext.tsx
Original file line number Diff line number Diff line change
@@ -1,9 +1,11 @@
import React, { createContext, useContext, useMemo } from 'react';
import { SkipClient } from '@skip-go/client';
import { SkipClient, SkipClientOptions } from '@skip-go/client';
import { OfflineDirectSigner } from '@cosmjs/proto-signing';
import { OfflineAminoSigner } from '@cosmjs/amino';

// Create the context
interface SkipContextType {
createClient: (options: any) => SkipClient;
createClient: (options: SkipClientOptions) => SkipClient;
}

const SkipContext = createContext<SkipContextType | undefined>(undefined);
Expand All @@ -15,15 +17,17 @@ interface SkipProviderProps {

export function SkipProvider({ children }: SkipProviderProps) {
const createClient = useMemo(() => {
return (options: any) => new SkipClient(options);
return (options: SkipClientOptions) => new SkipClient(options);
}, []);

return <SkipContext.Provider value={{ createClient }}>{children}</SkipContext.Provider>;
}

// Update the hook to accept getCosmosSigner
interface UseSkipClientOptions {
getCosmosSigner: () => Promise<any>;
getCosmosSigner: () => Promise<
OfflineAminoSigner | OfflineDirectSigner | (OfflineAminoSigner & OfflineDirectSigner)
>;
}

export function useSkipClient(options: UseSkipClientOptions) {
Expand Down

0 comments on commit 625ce3a

Please sign in to comment.