-
Notifications
You must be signed in to change notification settings - Fork 984
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
[#16094] Messaging - In Chat - Implement Options Menu for 1-1 and com… #16131
Conversation
:as chat} (rf/sub [:chats/current-chat]) | ||
display-name (if (= chat-type constants/one-to-one-chat-type) | ||
(first (rf/sub [:contacts/contact-two-names-by-identity chat-id])) | ||
(str emoji " " chat-name)) |
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.
nit - should this be an i18n label? 🤔
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.
its not my code, im not sure what's that, but doesn't look like label
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.
nice one @flexsurfer ! 🏄
Jenkins BuildsClick to see older builds (4)
|
@flexsurfer thanx for the PR. Please take a look at the following issues ISSUE 1 "No matching clause" exception on opening options menu in community channelSteps:
Actual result: app crashes with "No matching clause" exception. UI freezes, app needs to be restarted. telegram-cloud-document-2-5461147317404970033.mp4 |
ISSUE 2 Keyboard is not closed when Options bottom sheet is opened in chatSteps:
Actual result: keyboard is not closed. On Android bottom sheet appears above the keyboard, on IOS behind it. Android: telegram-cloud-document-2-5461147317404970042.mp4IOS: telegram-cloud-document-2-5461147317404970044.mp4 |
ISSUE 3 Deleting chat via Option menu does not result in closing currently opend chatSteps:
Expected result: user is redirected to Messages screen. Chat is deleted. Actual result: user remains on broken chat screen with "Add unknown to send message" info message. telegram-cloud-document-2-5461147317404970047.mp4 |
thank @pavloburykh fixed, for community chat i disabled options, should be implemented separately, we need to file separate issue with detailed description |
64% of end-end tests have passed
Not executed tests (5)Failed tests (10)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (18)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
|
72% of end-end tests have passed
Not executed tests (4)Failed tests (8)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Passed tests (21)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePR:
|
@flexsurfer thank you. PR is ready to be merged. Failed e2e are not PR related. |
fixes #16094