-
Notifications
You must be signed in to change notification settings - Fork 982
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
Hide empty categories and categories that include only hidden channels #20006
Conversation
Jenkins BuildsClick to see older builds (24)
|
88% of end-end tests have passed
Failed tests (5)Click to expandClass TestDeepLinksOneDevice:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestWalletMultipleDevice:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (46)Click to expandClass TestActivityMultipleDevicePRTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestWalletOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
|
:type :title | ||
:text-align :left | ||
:icon-name :i/close | ||
:on-press #(rf/dispatch [:navigate-back])}] | ||
[quo/empty-state | ||
{:image (resources/get-themed-image :cat-in-box theme) | ||
:description (when-not fetching? (i18n/label :t/here-is-a-cat-in-a-box-instead)) | ||
:title (if fetching? "Fetching community..." "Failed to fetch community") | ||
:title (if fetching? |
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.
Small additional change, leftover from #19968 (comment)
(should not affect review or e2e, just moved string to translation files)
Thanks for your work @Parveshdhull! |
97032f2
to
76672bc
Compare
✔️ status-mobile/prs/android/PR-20006#3 🔹 ~7 min 22 sec 🔹 97032f2 🔹 📦 android package |
76672bc
to
720b1fe
Compare
Not sure why unit-tests are failing after rebase. @siddarthkay Please can you check. |
720b1fe
to
e1c8443
Compare
Maybe because I pressed restart tests button, but now it seems all tests are failing |
Hey @Parveshdhull : will take a look ASAP |
awsm, thank you very much |
#19994 PR merged now, without any issues 🤔 |
e1c8443
to
052813e
Compare
#20061 also does not have the same issue. so probably this issue is host related. |
hi, reverting #20044, did fix tests for my PR |
Hi @Parveshdhull |
cool, thank you |
This reverts commit 05f3810. Signed-off-by: Parvesh Monu <parvesh.dhullmonu@gmail.com>
b5d0169
to
1817a86
Compare
fixes #19885
status: ready