-
Notifications
You must be signed in to change notification settings - Fork 229
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
Whatsapp quote messages #1920
Whatsapp quote messages #1920
Conversation
…to get quote messages
Thank you @gfd2020! |
I get this template and make it work on iped. It is a html, css, javascript inner modal alert. Very cool. It has confirm options too ... |
Seems very nice! |
Good! I'm almost done implementing it. |
If removing it decreases the dialog size, I think we can stay without it, otherwise I would put a generic title like "Message" or "Warning".
Yes, please keep the copyright into the source if you copy it into the project, list the library in ThirdParty.txt file and copy its MIT license to licenses folder. PS: Another advantage of using a javascript solution instead of a callback into java code is less exposure to possible security issues. |
Just copy link to the project page on javascript and css files. |
Hi @lfcnassif , Sorry to bother you. Were you able to redo the tests for this PR? |
I'm really sorry @gfd2020, not yet, other things took my attention... I promise you the next new feature I'll focus on will be this (bug fixes may take priority). But I'll be very busy in the next days and weeks, including this weekend. I'll try to return to this next week in the night... |
Hi @lfcnassif . I see. Just take your time. No problem. I think removing bugs is more important. This PR will eventually be integrated at the right time. |
Hi @gfd2020. I'm merging this right now to avoid more merge conflicts with WA parser improvements @wladimirleite currently is working on. I detected just very minor differences between this and master and I think this is very very good as is. I'll try to redo my tests later in January to confirm if my findings are really an issue or not. Thank you very much for this great feature! |
Thanks @lfcnassif! @gfd2020, I just saw the generated HTML including the support to quote messages you implement, it looks really nice and will be very useful. Thank you! |
Implementation of WhatsApp quote messages.
Android Old DB - OK ( carved quotes fully implemented )
IPHONE - OK ( carved quotes NOT fully implemented )
Android New DB - OK ( carved quotes fully implemented )