Skip to content

Conversation

@jakobroehrl
Copy link
Contributor

@jakobroehrl jakobroehrl commented Dec 28, 2020

Signed-off-by: Jakob Röhrl [email protected]
solves: #2800
image

works only in group chats, not for guest and my own messages
continuation of #4477

@nickvergessen
Copy link
Member

we actually need to find a way, because atm private replies don't display in the messages list

As said before, this will not work and I will not make it work. We will not be able to show messages of another chat 1 in the private chat 2. Checking if users have/retain access and so on is way to difficult. That's why I posted it in #4477 (comment)

If you reply, we need to copy it somehow into the chat input, either with:

> Original message

Private reply

or

Original author: "Original message"

Private reply

or something.

@szaimen
Copy link
Contributor

szaimen commented Jan 5, 2021

> Original message

Private reply

Sound good to me. This is e.g. how Whatsapp handles this. Of course it would need to display the whole message in quotes in this case and not a shortened one.

@nickvergessen nickvergessen marked this pull request as draft January 7, 2021 15:38
@jakobroehrl jakobroehrl force-pushed the enh/privateReply2 branch 5 times, most recently from 2c3d8db to 1ddc63f Compare February 23, 2021 13:57
@jakobroehrl jakobroehrl changed the title Private reply 2 Private reply without quote Feb 23, 2021
@jakobroehrl
Copy link
Contributor Author

@nickvergessen
I think this looks good, could you look over it?

jakobroehrl and others added 4 commits February 24, 2021 09:07
Signed-off-by: Jakob Röhrl <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
@jakobroehrl jakobroehrl marked this pull request as ready for review February 24, 2021 08:21
@nickvergessen nickvergessen merged commit 62aae8a into master Feb 24, 2021
@nickvergessen nickvergessen deleted the enh/privateReply2 branch February 24, 2021 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants