<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Sat, Dec 31, 2016 at 12:49 AM Daniel Berlin <<a href="mailto:dberlin@dberlin.org">dberlin@dberlin.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg">On Fri, Dec 30, 2016 at 3:36 PM, Mehdi Amini <span dir="ltr" class="gmail_msg"><<a href="mailto:mehdi.amini@apple.com" class="gmail_msg" target="_blank">mehdi.amini@apple.com</a>></span> wrote:<br class="gmail_msg"><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class="gmail_msg"><div class="gmail_msg"><div class="m_-7499801369921333899h5 gmail_msg"><br class="gmail_msg"><div class="gmail_msg"><blockquote type="cite" class="gmail_msg"><div class="gmail_msg">On Dec 30, 2016, at 3:26 PM, Daniel Berlin <<a href="mailto:dberlin@dberlin.org" class="gmail_msg" target="_blank">dberlin@dberlin.org</a>> wrote:</div><br class="m_-7499801369921333899m_-1612719899131616300Apple-interchange-newline gmail_msg"><div class="gmail_msg"><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">PS: consider sending replies in phabricator, because email replies sometimes doesn't appear there (like this one)</div></div></div></div></blockquote><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">I'm strongly against that, in the sense that phabricator needs to integrate with existing workflows.  I do real reviews in phab, but just overall commenting, don't have time or energy to click when i'm reading email :)</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">If email to phabricator is not currently working, we should fix that.</div><div class="gmail_msg">(I believe it is *supposed to be working*)</div></div></div></div></div></blockquote></div><br class="gmail_msg"></div></div><div class="gmail_msg">Inline answer in emails never worked in Phab. AFAIK.</div></div></blockquote><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg"><br class="gmail_msg"></div></div></div></div><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><div class="gmail_msg">Sigh.</div></div></div></div><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><div class="gmail_msg"> </div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class="gmail_msg"><div class="gmail_msg">It use to be that top answers were included as comment. But even that was not in Phab upstream but in our instance only (IIUC), so it can easily have been lost with any update.</div></div></blockquote></div></div></div><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><div class="gmail_msg">That seems super annoying.</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Even bugzilla can get this right :(</div></div></div></div></blockquote><div><br></div><div>Generally, only top-posting has worked in replies to update phab. Parsing in-line replies across different email clients is a PITA, unfortunately, so unless we find somebody who'd have fun hacking that up, it's unlikely to happen.</div><div><br></div><div>Currently, incoming email parsing is also completely broken, because the https switch invalidated our sendgrid config, and sendgrid now wants us to jump through hoops to update the info...</div></div></div>