-
Notifications
You must be signed in to change notification settings - Fork 985
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
[17377-17378] Fix reply view in activity center showing only one phot… #17388
Conversation
Jenkins BuildsClick to see older builds (38)
|
src/quo2/components/notifications/activity_logs_photos/view.cljs
Outdated
Show resolved
Hide resolved
src/status_im2/contexts/shell/activity_center/notification/reply/view.cljs
Outdated
Show resolved
Hide resolved
src/status_im2/contexts/shell/activity_center/notification/reply/view.cljs
Outdated
Show resolved
Hide resolved
84% of end-end tests have passed
Failed tests (7)Click to expandClass TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Passed tests (36)Click to expandClass TestActivityMultipleDevicePR:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
|
src/status_im2/contexts/shell/activity_center/notification/reply/view.cljs
Outdated
Show resolved
Hide resolved
src/status_im2/contexts/shell/activity_center/notification/reply/view.cljs
Outdated
Show resolved
Hide resolved
src/quo2/components/notifications/activity_logs_photos/view.cljs
Outdated
Show resolved
Hide resolved
@ilmotta @smohamedjavid |
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.
f566e0e
to
03873c5
Compare
src/status_im2/contexts/shell/activity_center/notification/reply/view.cljs
Outdated
Show resolved
Hide resolved
src/status_im2/contexts/shell/activity_center/notification/reply/view.cljs
Outdated
Show resolved
Hide resolved
@ibrkhalil |
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.
Code LGTM 🚀
47% of end-end tests have passed
Failed tests (23)Click to expandClass TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityOneDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityMultipleDevicePRTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (20)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestCommunityOneDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
|
hi @ibrkhalil, thank you for PR. No issues from my side. PR is ready to be merged |
fixes #17377 #17378
Fixes an issue that happened when a user replied to another user in community channel, The activity logs component only rendered a single image (If they were multiple in the reply, Only one would show) and didn't show the text in the reply if it was with an image
Presumably, Might need a design review since it's an extension of the previoulsy implemented component
status: ready