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

chore: add sign and submit with passwoard #2663

Merged
merged 118 commits into from
Sep 15, 2023
Merged

Conversation

SorinC6
Copy link
Contributor

@SorinC6 SorinC6 commented Sep 6, 2023

@SorinC6 SorinC6 added wip Shows that a PR shouldn't be merge chore labels Sep 6, 2023
@stackchain stackchain added this to the 5.0.0 milestone Sep 7, 2023
@michaeljscript
Copy link
Collaborator

This one is still WIP, right?

@SorinC6
Copy link
Contributor Author

SorinC6 commented Sep 8, 2023

This one is still WIP, right?

Hi, yes! but the flow related to the sign and submit with password is almost ready

@SorinC6 SorinC6 added help wanted Extra attention is needed wip Shows that a PR shouldn't be merge labels Sep 14, 2023
@SorinC6 SorinC6 removed help wanted Extra attention is needed wip Shows that a PR shouldn't be merge labels Sep 14, 2023
@jorbuedo
Copy link
Contributor

@SorinC6 What's the status? What help do you need? Is it about the backend error?

@SorinC6
Copy link
Contributor Author

SorinC6 commented Sep 14, 2023

@SorinC6 What's the status? What help do you need? Is it about the backend error?

Hi @jorbuedo the PR is ready now, Krimo helped to fix the issue on submitTx

Copy link
Collaborator

@michaeljscript michaeljscript left a comment

Choose a reason for hiding this comment

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

Hello, I've noticed that when I try to place a limit order, the button is unresponsive. I've noticed the warning [Error: Protocol needs to be one of [minswap, wingriders, sundaeswap, muesliswap, muesliswap_v2]] in console, so maybe it's related. I've tried to Swap 1 ADA for 3,333,333,333 HOSKY at the limit price 0.0000000003

apps/wallet-mobile/src/features/Swap/common/mocks.ts Outdated Show resolved Hide resolved
@jorbuedo
Copy link
Contributor

Hello, I've noticed that when I try to place a limit order, the button is unresponsive. I've noticed the warning [Error: Protocol needs to be one of [minswap, wingriders, sundaeswap, muesliswap, muesliswap_v2]] in console, so maybe it's related. I've tried to Swap 1 ADA for 3,333,333,333 HOSKY at the limit price 0.0000000003

Following yesterday's conversation. I don't think we have any protocol validation on the app. So I assume this error came from the backend?

@SorinC6
Copy link
Contributor Author

SorinC6 commented Sep 14, 2023

Hello, I've noticed that when I try to place a limit order, the button is unresponsive. I've noticed the warning [Error: Protocol needs to be one of [minswap, wingriders, sundaeswap, muesliswap, muesliswap_v2]] in console, so maybe it's related. I've tried to Swap 1 ADA for 3,333,333,333 HOSKY at the limit price 0.0000000003

Following yesterday's conversation. I don't think we have any protocol validation on the app. So I assume this error came from the backend?

yeah, seems like for Spectrum protocol - some of the pairs have this as default have this issues that the protocol is not supported but as Jordi said we don't have yet a validation. We could create a ticket for this to be solved in other PR later

@@ -907,6 +909,7 @@ export const makeShelleyWallet = (constants: typeof MAINNET | typeof TESTNET) =>
}

async submitTransaction(signedTx: string) {
console.log('submitTransaction signedTx', signedTx)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
console.log('submitTransaction signedTx', signedTx)
console.log('submitTransaction signedTx', signedTx)

@@ -621,7 +621,7 @@ export const makeShelleyWallet = (constants: typeof MAINNET | typeof TESTNET) =>
}
}

async signTx(unsignedTx: YoroiUnsignedTx, decryptedMasterKey: string) {
async signTx(unsignedTx: YoroiUnsignedTx, decryptedMasterKey: string, datum?: any) {
Copy link
Member

Choose a reason for hiding this comment

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

update the type (ShelleyWallet + ByronWallet)

@@ -420,6 +421,7 @@ export const useSignWithHwAndSubmitTx = (
...options?.signTx,
onSuccess: (signedTx, args, context) => {
options?.signTx?.onSuccess?.(signedTx, args, context)
console.log('useSignWithHwAndSubmitTx', {signedTx, args, context})
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
console.log('useSignWithHwAndSubmitTx', {signedTx, args, context})
console.log('useSignWithHwAndSubmitTx', {signedTx, args, context})

Copy link
Member

Choose a reason for hiding this comment

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

I've updated it, can you revert, and reapply later if needed?

@stackchain stackchain merged commit 8cd4494 into develop Sep 15, 2023
@stackchain stackchain deleted the chore/sign-and-submit-tx branch September 15, 2023 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants