Ditto.
Michael S. MacFadden Director of Research and Development SOLUTE Consulting 4250 Pacific Highway, Suite 211 San Diego, CA 92110 Office: (619) 758-9900 macfadden.mich...@solute.us www.solute.us CONFIDENTIALITY NOTICE: This e-mail and any attachments are intended solely for the use of the individual or entity to whom it is addressed. It may contain information that is privileged, confidential and exempt, or protected from disclosure under applicable law. If the reader of this message is not the intended recipient or the employee or agent responsible for delivering it to the intended recipient, you are hereby notified that any review, use, disclosure, distribution, or copying of this communication is strictly prohibited. If you have received this communication in error, please notify the sender immediately and destroy all copies and attachments. On Sep 1, 2012, at 2:07 PM, Yuri Z <vega...@gmail.com> wrote: > Congrats on your first commit Ali! > > On Sat, Sep 1, 2012 at 10:38 PM, Ali Lown <a...@lown.me.uk> wrote: > >> >> >>> On Sept. 1, 2012, 5:45 a.m., Yuri Zelikov wrote: >>>> >> >> Committed as r1379829 >> >> >> - Ali >> >> >> ----------------------------------------------------------- >> This is an automatically generated e-mail. To reply, visit: >> https://reviews.apache.org/r/6777/#review10969 >> ----------------------------------------------------------- >> >> >> On Aug. 31, 2012, 9:11 p.m., Ali Lown wrote: >>> >>> ----------------------------------------------------------- >>> This is an automatically generated e-mail. To reply, visit: >>> https://reviews.apache.org/r/6777/ >>> ----------------------------------------------------------- >>> >>> (Updated Aug. 31, 2012, 9:11 p.m.) >>> >>> >>> Review request for wave, Michael MacFadden, Yuri Zelikov, and Vicente J. >> Ruiz Jurado. >>> >>> >>> Description >>> ------- >>> >>> Following on from vjrj's review request #6339 which has stalled, I >> present my patch for WAVE-181: >>> >>> This assumes that the blip's received time is the same as the sent time >> (a reasonable assumption for everything apart from a slow federation), and >> so displays the received time to the user instead of the 1/1/1970 due to >> the current deficiency in the c/s protocol. >>> >>> I am choosing to do this rather than attempt to fix the c/s protocol >> since the June discussions showed an interest in changing the code >> organisation (and possibly protocols) -> which would let us fix this >> properly. >>> Also, with this patch, as far as a 'user' is concerned WAVE-181 would be >> fixed. >>> At the least, it is better than simply hiding the date, since that >> renders it more difficult for a 'user' to determine the order of a >> discussion. >>> >>> Comments, and further discussion? >>> >>> >>> This addresses bug WAVE-181. >>> https://issues.apache.org/jira/browse/WAVE-181 >>> >>> >>> Diffs >>> ----- >>> >>> /src/org/waveprotocol/wave/client/StageTwo.java 1379094 >>> >> /src/org/waveprotocol/wave/client/wavepanel/render/UndercurrentShallowBlipRenderer.java >> 1379094 >>> >>> Diff: https://reviews.apache.org/r/6777/diff/ >>> >>> >>> Testing >>> ------- >>> >>> Works on my machine between 2 users on the same server. >>> Time discrepancies between this 'guessed' time, and the times recorded >> in a snapshot varied by a maximum of a minute. >>> >>> >>> Thanks, >>> >>> Ali Lown >>> >>> >> >>
