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 status-bar color for new ui #13641

Merged
merged 1 commit into from
Jul 13, 2022
Merged

fix status-bar color for new ui #13641

merged 1 commit into from
Jul 13, 2022

Conversation

Parveshdhull
Copy link
Member

fixes #13612

status: ready

@Parveshdhull Parveshdhull self-assigned this Jul 8, 2022
@status-github-bot status-github-bot bot added this to REVIEW in Pipeline for QA Jul 8, 2022
@Parveshdhull Parveshdhull mentioned this pull request Jul 8, 2022
@Parveshdhull Parveshdhull moved this from REVIEW to E2E Tests in Pipeline for QA Jul 8, 2022
@status-im-auto
Copy link
Member

status-im-auto commented Jul 8, 2022

Jenkins Builds

Click to see older builds (5)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 52abf51 #1 2022-07-08 20:08:55 ~10 min ios 📦ipa 📲
✔️ 52abf51 #1 2022-07-08 20:10:27 ~11 min android-e2e 📦apk 📲
✔️ 52abf51 #2 2022-07-08 21:01:26 ~9 min android 📦apk 📲
✔️ 6d7a8c3 #2 2022-07-11 21:08:22 ~9 min ios 📦ipa 📲
✔️ 6d7a8c3 #2 2022-07-11 21:10:02 ~11 min android-e2e 📦apk 📲
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 38e8b9f #3 2022-07-11 21:42:08 ~9 min ios 📦ipa 📲
✔️ 38e8b9f #4 2022-07-11 21:44:00 ~11 min android 📦apk 📲
✔️ 38e8b9f #4 2022-07-12 13:18:51 ~9 min android-e2e 📦apk 📲
✔️ 3ec0bc8 #5 2022-07-13 10:11:54 ~9 min android-e2e 📦apk 📲
✔️ 3ec0bc8 #5 2022-07-13 10:12:04 ~9 min android 📦apk 📲
✔️ 3ec0bc8 #4 2022-07-13 10:14:15 ~11 min ios 📦ipa 📲

@status-im-auto
Copy link
Member

96% of end-end tests have passed

Total executed tests: 84
Failed tests: 3
Passed tests: 81
IDs of failed tests: 702196,702194,702188 

Failed tests (3)

Click to expand
  • Rerun failed tests

  • 1. test_pairing_sync_contacts_block_unblock, id: 702196

    Device 2: Tap on found: Button
    Device 2: Find `Button` by `accessibility id`: `blocked-users-list-button`

    Device 2: Button by accessibility id: `blocked-users-list-button` is not found on the screen

    Class: TestPairingSyncMultipleDevicesMerged

    Device sessions

    2. test_pairing_sync_initial_contacts_blocked_users, id: 702194

    Device 2: Tap on found: Button
    Device 2: Find `Button` by `accessibility id`: `blocked-users-list-button`

    Device 2: Button by accessibility id: `blocked-users-list-button` is not found on the screen

    Class: TestPairingSyncMultipleDevicesMerged

    Device sessions

    3. test_cellular_settings_on_off_public_chat_fetching_history, id: 702188

    Device 1: Find `Button` by `xpath`: `//*[@text="Use mobile data"]/following-sibling::android.widget.Switch[1]`
    Device 1: Find `Button` by `xpath`: `//*[@text="Ask me when on mobile network"]/following-sibling::android.widget.Switch[1]`

    Chat history was not fetched with mobile data fetching ON [[flaky; issue when sometimes history is not fetched from offline for public chat, needs investigation]]

    Class: TestContactBlockMigrateKeycardMultipleSharedDevices

    Device sessions

    Passed tests (81)

    Click to expand

    1. test_wallet_manage_assets, id: 700758

    Class: TestWalletManagementDeviceMerged

    Device sessions

    2. test_public_chat_message_edit, id: 700734

    Class: TestPublicChatMultipleDeviceMerged

    Device sessions

    3. test_contact_block_unblock_public_chat_offline, id: 702176

    Class: TestContactBlockMigrateKeycardMultipleSharedDevices

    Device sessions

    4. test_1_1_chat_text_message_edit_delete_push_disappear, id: 695843

    Class: TestOneToOneChatMultipleSharedDevices

    Device sessions

    5. test_keycard_relogin_after_restore, id: 700768

    Class: TestKeycardTxOneDeviceMerged

    Device sessions

    6. test_group_chat_push_system_messages_when_invited, id: 3994

    Class: TestGroupChatMultipleDeviceMerged

    Device sessions

    7. test_ens_command_send_tx_eth_1_1_chat, id: 702153

    Class: TestEnsStickersMultipleDevicesMerged

    Device sessions

    8. test_send_tx_token_7_decimals, id: 700764

    Class: TestSendTxDeviceMerged

    Device sessions

    9. test_contact_add_remove_mention_default_username_nickname_public_chat, id: 702175

    Class: TestContactBlockMigrateKeycardMultipleSharedDevices

    Device sessions

    10. test_public_chat_open_using_deep_link, id: 700739

    Class: TestPublicChatBrowserOneDeviceMerged

    Device sessions

    11. test_public_chat_unread_messages_counter, id: 5360

    Class: TestPublicChatMultipleDeviceMerged

    Device sessions

    12. test_1_1_chat_command_decline_eth_push_changing_state, id: 6265

    Class: TestCommandsMultipleDevicesMerged

    Device sessions

    13. test_public_chat_emoji_send_copy_paste_reply, id: 700719

    Class: TestPublicChatMultipleDeviceMerged

    Device sessions

    14. test_1_1_chat_command_request_and_send_tx_stt_in_1_1_chat_offline, id: 6263

    Class: TestCommandsMultipleDevicesMerged

    Device sessions

    15. test_browser_bookmarks_create_edit_remove, id: 702077

    Class: TestPublicChatBrowserOneDeviceMerged

    Device sessions

    16. test_send_tx_eth_check_logcat, id: 700763

    Class: TestSendTxDeviceMerged

    Device sessions

    17. test_public_chat_unread_messages_counter_for_mention_relogin, id: 700718

    Class: TestPublicChatMultipleDeviceMerged

    Device sessions

    18. test_send_tx_custom_token_18_decimals_invalid_password, id: 700765

    Class: TestSendTxDeviceMerged

    Device sessions

    19. test_onboarding_add_new_multiaccount_username_by_position_pass_validation, id: 700747

    Class: TestOnboardingOneDeviceMerged

    Device sessions

    20. test_wallet_add_account_seed_phrase_validation, id: 700762

    Class: TestWalletManagementDeviceMerged

    Device sessions

    21. test_public_chat_delete_chat_long_press, id: 5319

    Class: TestPublicChatMultipleDeviceMerged

    Device sessions

    22. test_onboarding_home_initial_popup, id: 700742

    Class: TestOnboardingOneDeviceMerged

    Device sessions

    23. test_browser_blocked_url, id: 702072

    Class: TestPublicChatBrowserOneDeviceMerged

    Device sessions

    24. test_wallet_add_hide_unhide_account_private_key, id: 700761

    Class: TestWalletManagementDeviceMerged

    Device sessions

    25. test_1_1_chat_message_reaction, id: 6315

    Class: TestOneToOneChatMultipleSharedDevices

    Device sessions

    26. test_ens_mention_push_highlighted_public_chat, id: 702156

    Class: TestEnsStickersMultipleDevicesMerged

    Device sessions

    27. test_1_1_chat_emoji_send_reply_and_open_link, id: 5373

    Class: TestOneToOneChatMultipleSharedDevices

    Device sessions

    28. test_keycard_send_tx_sign_message_request_stt_testdapp, id: 700769

    Class: TestKeycardTxOneDeviceMerged

    Device sessions

    29. test_1_1_chat_image_send_save_reply, id: 6305

    Class: TestOneToOneChatMultipleSharedDevices

    Device sessions

    30. test_group_chat_highligted, id: 5756

    Class: TestGroupChatMultipleDeviceMerged

    Device sessions

    31. test_public_chat_link_send_open, id: 700736

    Class: TestPublicChatMultipleDeviceMerged

    Device sessions

    32. test_keycard_command_send_tx_eth_1_1_chat, id: 702186

    Class: TestContactBlockMigrateKeycardMultipleSharedDevices

    Device sessions

    33. test_onboarding_share_contact_address, id: 700743

    Class: TestOnboardingOneDeviceMerged

    Device sessions

    34. test_sticker_1_1_public_chat, id: 702157

    Class: TestEnsStickersMultipleDevicesMerged

    Device sessions

    35. test_onboarding_backup_seed_phrase_restore_same_login_logcat, id: 700745

    Class: TestOnboardingOneDeviceMerged

    Device sessions

    36. test_restore_set_up_wallet_sign_phrase, id: 700749

    Class: TestRestoreOneDeviceMerged

    Device sessions

    37. test_ens_mention_nickname_1_1_chat, id: 702155

    Class: TestEnsStickersMultipleDevicesMerged

    Device sessions

    38. test_1_1_chat_is_shown_message_sent_delivered_from_offline, id: 5310

    Class: TestOneToOneChatMultipleSharedDevices

    Device sessions

    39. test_browser_delete_close_tabs, id: 702076

    Class: TestPublicChatBrowserOneDeviceMerged

    Device sessions

    40. test_public_chat_mark_all_messages_as_read, id: 6270

    Class: TestPublicChatMultipleDeviceMerged

    Device sessions

    41. test_browser_connection_is_secure_not_secure_warning, id: 702073

    Class: TestPublicChatBrowserOneDeviceMerged

    Device sessions

    42. test_send_tx_set_recipient_options, id: 700757

    Class: TestSendTxDeviceMerged

    Device sessions

    43. test_onboarding_cant_sign_in_with_invalid_password_logcat, id: 700746

    Class: TestOnboardingOneDeviceMerged

    Device sessions

    44. test_send_tx_sign_message_2tx_in_batch_tx_filters_request_stt_testdapp, id: 5342

    Class: TestSendTxDeviceMerged

    Device sessions

    45. test_group_chat_join_send_text_messages_push, id: 700731

    Class: TestGroupChatMultipleDeviceMerged

    Device sessions

    46. test_browser_web3_permissions_testdapp, id: 702078

    Class: TestPublicChatBrowserOneDeviceMerged

    Device sessions

    47. test_1_1_chat_text_message_with_push, id: 6316

    Class: TestOneToOneChatMultipleSharedDevices

    Device sessions

    48. test_wallet_add_delete_watch_only_account, id: 700760

    Class: TestWalletManagementDeviceMerged

    Device sessions

    49. test_public_chat_tag_message, id: 700738

    Class: TestPublicChatBrowserOneDeviceMerged

    Device sessions

    50. test_onboarding_share_wallet_address, id: 700744

    Class: TestOnboardingOneDeviceMerged

    Device sessions

    51. test_keycard_create_account_unlock_same_seed, id: 5689

    Class: TestKeycardTxOneDeviceMerged

    Device sessions

    52. test_1_1_chat_command_send_tx_eth_outgoing_tx_push, id: 6253

    Class: TestCommandsMultipleDevicesMerged

    Device sessions

    53. test_public_chat_navigate_to_chat_when_relaunch, id: 5396

    Class: TestPublicChatBrowserOneDeviceMerged

    Device sessions

    54. test_pairing_sync_1_1_chat_message, id: 702198

    Class: TestPairingSyncMultipleDevicesMerged

    Device sessions

    55. test_pairing_sync_public_chat_add_remove, id: 702199

    Class: TestPairingSyncMultipleDevicesMerged

    Device sessions

    56. test_public_chat_copy_and_paste_message_in_chat_input, id: 5317

    Class: TestPublicChatBrowserOneDeviceMerged

    Device sessions

    57. test_public_chat_fetch_more_history, id: 5675

    Class: TestPublicChatBrowserOneDeviceMerged

    Device sessions

    58. test_wallet_fetching_balance_after_offline_insufficient_funds_errors, id: 700766

    Class: TestWalletManagementDeviceMerged

    Device sessions

    59. test_group_chat_offline_pn, id: 3998

    Class: TestGroupChatMultipleDeviceMerged

    Device sessions

    60. test_restore_account_migrate_multiaccount_to_keycard_db_saved, id: 702177

    Class: TestContactBlockMigrateKeycardMultipleSharedDevices

    Device sessions

    61. test_group_chat_add_new_member_activity_centre, id: 700732

    Class: TestGroupChatMultipleDeviceMerged

    Device sessions

    62. test_public_chat_links_with_previews_github_youtube_twitter_gif_send_enable, id: 700737

    Class: TestPublicChatMultipleDeviceMerged

    Device sessions

    63. test_ens_purchased_in_profile, id: 702152

    Class: TestEnsStickersMultipleDevicesMerged

    Device sessions

    64. test_1_1_chat_non_latin_message_to_newly_added_contact_with_profile_picture_on_different_networks, id: 5315

    Class: TestOneToOneChatMultipleSharedDevices

    Device sessions

    65. test_keycard_send_tx_eth, id: 700767

    Class: TestKeycardTxOneDeviceMerged

    Device sessions

    66. test_restore_account_migrate_multiaccount_to_keycard_no_db_saved_add_wallet_send_tx, id: 702189

    Class: TestRestoreOneDeviceMerged

    Device sessions

    67. test_browser_offline, id: 702075

    Class: TestPublicChatBrowserOneDeviceMerged

    Device sessions

    68. test_browser_invalid_url, id: 702074

    Class: TestPublicChatBrowserOneDeviceMerged

    Device sessions

    69. test_restore_seed_phrase_field_validation, id: 700750

    Class: TestRestoreOneDeviceMerged

    Device sessions

    70. test_wallet_add_account_generate_new, id: 700759

    Class: TestWalletManagementDeviceMerged

    Device sessions

    71. test_start_new_chat_public_key_validation, id: 702158

    Class: TestEnsStickersMultipleDevicesMerged

    Device sessions

    72. test_restore_uppercase_whitespaces_seed_phrase_special_char_passw_logcat, id: 700748

    Class: TestRestoreOneDeviceMerged

    Device sessions

    73. test_public_chat_message_send_check_timestamps_while_on_different_tab, id: 5313

    Class: TestPublicChatMultipleDeviceMerged

    Device sessions

    74. test_wallet_tx_history_copy_tx_hash_on_cellular, id: 700756

    Class: TestWalletManagementDeviceMerged

    Device sessions

    75. test_1_1_chat_delete_via_delete_button_relogin, id: 5387

    Class: TestOneToOneChatMultipleSharedDevices

    Device sessions

    76. test_group_chat_leave_relogin, id: 3997

    Class: TestGroupChatMultipleDeviceMerged

    Device sessions

    77. test_public_chat_message_delete, id: 700735

    Class: TestPublicChatMultipleDeviceMerged

    Device sessions

    78. test_keycard_wallet_recover_pairing_check_balance_after_offline_tx_history, id: 700770

    Class: TestKeycardTxOneDeviceMerged

    Device sessions

    79. test_1_1_chat_push_emoji, id: 6283

    Class: TestOneToOneChatMultipleSharedDevices

    Device sessions

    80. test_pairing_sync_initial_public_chats, id: 702195

    Class: TestPairingSyncMultipleDevicesMerged

    Device sessions

    81. test_pairing_sync_contacts_add_remove_set_nickname_ens, id: 702197

    Class: TestPairingSyncMultipleDevicesMerged

    Device sessions

    @qoqobolo qoqobolo moved this from E2E Tests to IN TESTING in Pipeline for QA Jul 11, 2022
    @qoqobolo qoqobolo self-assigned this Jul 11, 2022
    @qoqobolo qoqobolo moved this from IN TESTING to REVIEW in Pipeline for QA Jul 11, 2022
    @Parveshdhull Parveshdhull force-pushed the fix/status-bar-color branch 2 times, most recently from 6d7a8c3 to 38e8b9f Compare July 11, 2022 21:32
    @qoqobolo qoqobolo moved this from REVIEW to IN TESTING in Pipeline for QA Jul 12, 2022
    @qoqobolo
    Copy link
    Contributor

    qoqobolo commented Jul 13, 2022

    @Parveshdhull, thanks for PR!
    Noticed one thing in the status bar behaviour:

    video_2022-07-13_12-10-13.mp4

    If I open the chat from the activity center the status bar is light grey, but when I open it from the home screen it's white in the chat view.
    Is it expected for the time being?

    @Parveshdhull
    Copy link
    Member Author

    Hi @qoqobolo, Thank you very much for testing the PR and finding this issue.

    So, for opening of the chat from the activity center notification, we are still using the navigation1 method, and in navigation1 we didn't pass the status bar color for each screen because it was the same as the home view, all white in light mode and black in dark mode.

    But for redesign, we need to pass the status-bar color for each screen separately because as you can see in screenshots, the status bar in the home is light grey while in chat it is white.
    image
    image

    Once the redesign of the activity center will be complete, we will migrate it to navigation 2 and pass the status-bar color accordingly, and that will fix this issue.

    @qoqobolo
    Copy link
    Contributor

    @Parveshdhull got it! Thanks for detailed clarification as usual and sorry for the endless questions what is expected and what is not 🙏

    @Parveshdhull
    Copy link
    Member Author

    @Parveshdhull got it! Thanks for detailed clarification as usual and sorry for the endless questions what is expected and what is not

    No problem :), You are right to ask the question, because It is still a bug. It is just that it will be fixed later.

    @qoqobolo qoqobolo moved this from IN TESTING to MERGE in Pipeline for QA Jul 13, 2022
    @Parveshdhull Parveshdhull merged commit 08194b3 into develop Jul 13, 2022
    Pipeline for QA automation moved this from MERGE to DONE Jul 13, 2022
    @Parveshdhull Parveshdhull deleted the fix/status-bar-color branch July 13, 2022 11:42
    chrispader pushed a commit to chrispader/status-mobile that referenced this pull request Jul 13, 2022
    chrispader pushed a commit to chrispader/status-mobile that referenced this pull request Aug 1, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    Archived in project
    Development

    Successfully merging this pull request may close these issues.

    Switcher sometimes shows glitched bar
    4 participants