-
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
Feature/allow user to select a color for the group chat #18982 #19088
Feature/allow user to select a color for the group chat #18982 #19088
Conversation
Jenkins BuildsClick to see older builds (57)
|
8f25e1b
to
4dc0e37
Compare
4f2860c
to
e48196e
Compare
934cd99
to
dbe0a31
Compare
This comment was marked as resolved.
This comment was marked as resolved.
dbe0a31
to
1ae01ce
Compare
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
hi @flexsurfer, |
This comment was marked as resolved.
This comment was marked as resolved.
cf845ad
to
85c2ad1
Compare
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.
here's the design review :)
b4f379b
to
ef503e4
Compare
@Francesca-G thank you , all feedback addressed cc @qoqobolo |
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.
@flexsurfer here's the review
thank you 😊
thank you @Francesca-G |
@flexsurfer there is a new issue ISSUE 8: [Android] Error and crash when adding an emoji to the namevideo_2024-03-21_13-42-03.mp4 |
9082dc9
to
0d0a324
Compare
thank you @qoqobolo fixed |
Thank you @flexsurfer, and hopefully last one: ISSUE 9: The
|
0d0a324
to
8d15f30
Compare
thank you @qoqobolo fixed |
19% of end-end tests have passed
Failed tests (38)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestActivityMultipleDevicePR:
Class TestDeepLinksOneDevice:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityOneDeviceMerged:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (9)Click to expandClass TestActivityMultipleDevicePRTwo:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
|
E2e failed because of the known random error, won't rerun it because of the queue. |
fixes #18982