-
Notifications
You must be signed in to change notification settings - Fork 983
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
Update status-go to develop-g607954bf #3236
Conversation
@adambabik I'm not sure what exactly needs to be reviewed on status-react side. As far as i see from changelog, no API (interface to status-react) breaking changes were made, except maybe this
But as far as i could tell, it's internal only change. TransactionResult JSON returned to status-react seems to be the same. |
Branch: PR-3236 |
@dmitryn No need to review anything. I just want to keep status-go up-to-date in status-react. Otherwise, if we plan to update after too many changes it will be hard to figure out potential issues. The important thing is geth 1.7.3 update. There is a possibility that we have not enough geth 1.7.3 nodes in our cluster so we would need to update them before this PR is merged. We should thoroughly test all scenarios like sending messages between users etc. |
@adambabik so I can merge this ? |
@yenda shouldn't it be tested by the Q&A team before merging? I am afraid it may not work in some configuration because of not enough geth 1.7.3 nodes. I will take a look what we have in our cluster first. |
27d01a1
to
5219171
Compare
@adambabik @oskarth what do you think about updating it a bit further to |
@mandrigin Would rather not expand scope since this will be in release which is already delayed and this PR in testing. Would you mind creating a separate PR for that update? |
@oskarth makes sense, I'll do that. |
Automated test results:test_send_eth_from_wallet_sign_now:x:Test Steps & Error message:
test_send_transaction_from_daap:white_check_mark::Test Steps & Error message:
test_send_eth_from_wallet_sign_later:white_check_mark::Test Steps & Error message:
test_transaction_send_command_one_to_one_chat:white_check_mark::Test Steps & Error message:
test_transaction_send_command_wrong_password:white_check_mark::Test Steps & Error message:
test_transaction_send_command_group_chat:x:Test Steps & Error message:
test_send_stt_from_wallet_via_enter_contact_code:white_check_mark::Test Steps & Error message:
test_send_eth_to_request_from_wallet:x:Test Steps & Error message:
test_send_eth_to_request_in_one_to_one_chat:x:Test Steps & Error message:
test_send_eth_to_request_in_group_chat:white_check_mark::Test Steps & Error message:
|
Branch: PR-3236 |
So i have checked basic functionality, like transacting, messaging , i was able to faucet some ETH and add / remove contacts, chatting in a group and public chat. i would say its enough to merge it. |
5219171
to
8a05aab
Compare
Signed-off-by: Eric Dvorsak <eric@dvorsak.fr>
8a05aab
to
65fc4ab
Compare
Another update after a bunch of refactoring we did recently.
Changelog (curated):