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
[DDW-414] Bump electron and related packages #2206
Conversation
Hi @yakovkaravelov. Here is the summary for build 14720. New issues 2 - When attempting to send a transaction that is not possible because I do not have enough funds an incorrect error message "Cannot calculate fees while there are pending transactions" is displayed. There are no pending transactions in the wallet. See here 3 - There is an also an error in the console "State snapshot log file creation failed". This is caused by downloading logs. The logs downloaded succesfully success message is still shown. See here 4 - Not able to run Daedalus on Linux. Attempted with both MainNet and TestNet. The only log available is this see log Test Document |
Hi @yakovkaravelov. I added two more items to my comment above and I updated the test summary in the PR Description |
…t-output-hk/daedalus into chore/ddw-414-bump-electron-package
Hi @yakovkaravelov . Here is the QA Summary for build 14590. Still Pending Issues. I was able to reproduce the 2 issues below on the new build 1 - On Mac and Windows on All Networks. There is an error in the console "State snapshot log file creation failed". This is caused by downloading logs. The logs downloaded succesfully success message is still shown. See here.I cannot test Linux as it will 2 - Not able to run Daedalus on Linux. Attempted with both MainNet and TestNet. The only log available is this see log *Fixed * Not an Isssue for this PR Test Document |
@ManusMcCole does this happen on all networks (Mainnet, Testnet, Shelley QA, Flight...)? cc @yakovkaravelov |
Hi @yakovkaravelov and @nikolaglumac. Build 14950.For "1 - On Mac and Windows.There is an error in the console "State snapshot log file creation failed". This is caused by downloading logs. The logs downloaded succesfully success message is still shown. See here" this issue happens on all Networks on Windows and Mac. I cannot test Linux as it will not launch Also for "2 - When attempting to send a transaction that is not possible because I do not have enough funds an incorrect error message "Cannot calculate fees while there are pending transactions" is displayed. There are no pending transactions in the wallet. See here" it was marked as fixed but today now I notice sometimes there is a delay where Daedalus takes a good few seconds to realise the user does not have enough UTXO for the transaction. The message calculating fees seems to have higher precedence over the "not enough ada.Try sending a smaller amount". See video from newest build. Im adding this to the report now |
@ManusMcCole I seriously doubt this is specific to this PR. Please test on other builds (2.4.0 release or develop builds) before adding this to the list of issues of this PR. |
Hi @yakovkaravelov . I have retested on build 15000 on all 3 platforms. Could replicate all the pending issues as listed in "Testing Summary" please see here |
…age' into chore/ddw-414-bump-electron-package
@miorsufianiohk @gabriela-ponce @ManusMcCole @danielmain when can we expect this one to be approved? |
@nikolaglumac So far the regression is ok. I found a couple of issues that still reproduce from previous builds on macOS:
@miorsufianiohk can you please confirm if you reproduce this in Ledger Nano X? |
@gabriela-ponce @miorsufianiohk @ManusMcCole @dmitrii-gaico please re-test all HW scenarios on all platforms and lets get some numbers 🙏 |
|
@nikolaglumac @danielmain Just finished checking the Ledger S cases on build 19005 for all platforms, and these are the findings:
|
@gabriela-ponce is the same issue reproducible on develop or latest release (4.3.1)? |
|
We have validated both scenarios at MacOS Catalina/BigSur | 4.3.1 current release and latest develop. In all cases issues are NOT reproducible. |
Perfect 👌 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A true masterpiece 💯
Amazing work @danielmain 🎉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Tested on 19020 Testnet with Nano S on Win, BigSur, Catalina and Linux 👍
This PR updated electron and related packages.
Jira Ticket
Latest test document
Regression Spreadsheet
Previous test documents
Regression Spreadsheet 2
Regression Spreadsheet 1
HW Regression Spreadsheet
OLD test document see here
Todos
Testing Checklist
App
Storybook
Testing Summary
QA Summary for build 18923:
QA Summary for build 18923:
New
[macOS] The error "transaction confirmation failed" is displayed if you create a transaction for Ledger S and connect the device after. Check this screenshot.
[Windows] Pair a Ledger S device, wait for it to sync and unlock it. Create a transaction without connecting the device, you'll see that Daedalus asks to "Run cardano app" instead of "Connect the Ledger S". A lot of errors are visible in the console. Check this screenshot.
Note: If you connect the device at this stage, you still can confirm the transaction.
[Intermittent - Windows] When trying to delete a Ledger S wallet, an error on the console is displayed. The process of deletion can take a while and if you restart Daedalus, the wallet is still there. Check this screenshot to see the error.
Pending
Fixed
I was able to pair Nano S with Mac Catalina, but wasn't able to execute any transaction. On Daedalus Transaction screen displayed "Export the public key on your device" but Nano S device displayed "Waiting for commands"
Daedalus crashed when I attempted to send ADA but the balance wasn't enough to cover transaction fee. I managed to duplicate the issue with other wallets too (both Byron and Shelley). Please see video
The "Blank screen fix" issue is ok now, but the state is not persisted after closing Daedalus. To clarify: In this build If you choose "Blank screen fix", when you close and reopen Daedalus, it is off again. In the version from production, it's a state that doesn't change until you deactivate it.
After "Blank screen fix" is active, it can't be deactivated. Check this video. Happens on all platforms. Reported here
Error on BigSur when closing Daedalus. Check screenshot. (Reported via Slack)
Each time I close Daedalus it reopens by itself. Happens on all platforms. Check this video. Reported here.
Daedalus restarts after choosing "Blank screen fix", but the fix is disabled (it should look like this). Check this video. Reported here.
Can't pair Ledger devices on macOS and Windows. Reported here, here, here and here.
Daedalus doesn't restart after selecting "Blank screen fix". Reported here.
When you select "Blank screen fix", the Cardano node stops, but Daedalus doesn't restart. If you close it manually after this, it would restart, but the "Blank screen fix" won't be enabled. Check this video. Reported here.
Review Checklist
Basics
feature
/bug
/chore
,release-x.x.x
)yarn test
)yarn dev
)yarn package
/ CI builds)yarn flow:test
)yarn lint
)yarn prettier:check
)yarn manage:translations
produces no changes)yarn storybook
)yarn.lock
file is updatedCode Quality
Testing
After Review
done
column on the YouTrack board