Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Make it possible to view read receipts #5367

Closed
alya opened this issue Apr 27, 2022 · 1 comment · Fixed by #5489
Closed

Make it possible to view read receipts #5367

alya opened this issue Apr 27, 2022 · 1 comment · Fixed by #5489
Assignees
@chrisbobbe
Labels
P1 high-priority server release goal Things we should try to coordinate with a major Zulip Server release. webapp parity Features that exist in the webapp that we need to port to mobile. We aren't aiming for full parity.

Comments

@alya
Copy link
Collaborator

alya commented Apr 27, 2022

In zulip/zulip#3618 / zulip/zulip#18935, we're making it possible to view read receipts in the webapp. We should add an option to the long-press message menu to make this possible on mobile as well.

This feature should of course respect the user privacy setting for whether to allow others to view read receipts from them. Where possible, it should follow the webapp design in terms of headings, sorting, etc.
@alya alya added P1 high-priority webapp parity Features that exist in the webapp that we need to port to mobile. We aren't aiming for full parity. server release goal Things we should try to coordinate with a major Zulip Server release. labels Apr 27, 2022
@timabbott
Copy link
Member

Just as a note on the privacy setting -- that's implemented server-side, so there should be nothing the app needs to do there. (Specifically, the server will just not send the client information about readers who have disabled sharing read receipts).

@neiljp neiljp mentioned this issue Aug 18, 2022
Open
chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Sep 15, 2022
@chrisbobbe
api: Add getReadReceipts binding (GET messages/{message_id}/read_rece…
e3b519a 
…ipts)

We'll use this for zulip#5367, "Make it possible to view read receipts".
chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Sep 15, 2022
@chrisbobbe
ReadReceiptsScreen: Add, with TODO to show data state more visibly
185e421 
This makes it possible to view read receipts.

Fixes: zulip#5367
@chrisbobbe chrisbobbe mentioned this issue Sep 15, 2022
Merged
chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Sep 16, 2022
@chrisbobbe
api: Add getReadReceipts binding (GET messages/{message_id}/read_rece…
626bba8 
…ipts)

We'll use this for zulip#5367, "Make it possible to view read receipts".
chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Sep 16, 2022
@chrisbobbe
ReadReceiptsScreen: Add, with TODO to show data state more visibly
dcce0e8 
This makes it possible to view read receipts.

Fixes: zulip#5367
chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Oct 19, 2022
@chrisbobbe
api: Add getReadReceipts binding (GET messages/{message_id}/read_rece…
bc930f6 
…ipts)

We'll use this for zulip#5367, "Make it possible to view read receipts".
chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Oct 19, 2022
@chrisbobbe
ReadReceiptsScreen: Add, with TODO to show data state more visibly
adc04bf 
This makes it possible to view read receipts.

Fixes: zulip#5367
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chrisbobbe chrisbobbe

Labels
P1 high-priority server release goal Things we should try to coordinate with a major Zulip Server release. webapp parity Features that exist in the webapp that we need to port to mobile. We aren't aiming for full parity.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add UI to view read receipts chrisbobbe/zulip-mobile
4 participants
@gnprice @alya @timabbott @chrisbobbe