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

Fix/lw 11758 error not displayed setting collateral with insufficient balance #1493

Conversation

mchappell
Copy link
Collaborator

@mchappell mchappell commented Oct 28, 2024

Checklist

  • JIRA - LW-11758
  • Proper tests implemented
  • Screenshots added.

Proposed solution

Correctly await the response from setting collateral before continuing rest of the chained functions

Testing

Notes added to linked ticket

Screenshots

Attach screenshots here if implementation involves some UI changes

@mchappell mchappell requested a review from a team as a code owner October 28, 2024 18:14
@pczeglik-iohk
Copy link
Contributor

pczeglik-iohk commented Oct 28, 2024

Allure Report

allure-report-publisher generated test report!

processReports: ✅ test report for 2defe29e

passed failed skipped flaky total result
Total 32 0 4 0 36

Copy link
Contributor

@mirceahasegan mirceahasegan left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link

sonarcloud bot commented Oct 29, 2024

@mchappell mchappell enabled auto-merge (squash) October 29, 2024 14:54
@mchappell mchappell merged commit 6b5aac4 into main Oct 29, 2024
30 checks passed
@mchappell mchappell deleted the fix/lw-11758-error-not-displayed-setting-collateral-with-insufficient-balance branch October 29, 2024 15:06
mchappell added a commit that referenced this pull request Oct 29, 2024
…ufficient balance (#1493)

* fix(nami): reset collateral data on modal close

* fix(nami): correctly await setting collateral

* fix(nami): init collateral explicitly checks available utxos

* fix(nami): clear pw secrets on collateral modal close

* fixup! fix(nami): init collateral explicitly checks available utxos
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.

4 participants