-
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
Add push notification on group chat invite #12440
Conversation
Jenkins BuildsClick to see older builds (35)
|
953bd59
to
4e2e77e
Compare
7540ffa
to
9843660
Compare
2747806
to
a11f089
Compare
77% of end-end tests have passed
Failed tests (16)Click to expand
Passed tests (53)Click to expand |
12% of end-end tests have passed
Failed tests (14)Click to expand
Passed tests (2)Click to expand
|
@briansztamfater thanks for your work! ISSUE 1: If UserA invites UserB and UserC to the group chat, and UserB joins the group, then on the UserC's device the invitation notification from UserA is displayed as an invitation from UserBSteps:
At the beginning of the video, PN from Keen Courageous Bongo turns into PN from Little Knobby Bear as he joins the group video_2021-09-15_17-25-18.mp4 |
Also, confirm that the issue for iOS will be logged separately, so now only PNs on Android should be fixed in the current PR. |
f0ee25e
to
580913a
Compare
Thanks @qoqobolo, already working on a fix for that issue. |
a5daa37
to
61ce927
Compare
@qoqobolo issue should be fixed now |
@briansztamfater thank you! No other issues were found. |
61ce927
to
de74da6
Compare
de74da6
to
1ab9956
Compare
@qoqobolo Just merged the |
@briansztamfater yes, sure |
0% of end-end tests have passed
Failed tests (25)Click to expand
|
0% of end-end tests have passed
Failed tests (25)Click to expand
|
0% of end-end tests have passed
Failed tests (1)Click to expand
|
100% of end-end tests have passed
Passed tests (25)Click to expand
|
@briansztamfater looks good to me, let's merge! |
Signed-off-by: Brian Sztamfater <brian@status.im>
1ab9956
to
5c0102c
Compare
fixes #12330
Summary
This PR adds an OS notification when a contact invites you to a group chat
Platforms
Functional
Steps to test
status: ready