incubator-wave-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ali Lown" <>
Subject Review Request: Assume receivedTime = sentTime for blip display
Date Sun, 26 Aug 2012 20:59:05 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for wave, Michael MacFadden, Yuri Zelikov, and Vicente J. Ruiz Jurado.


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.





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.


Ali Lown

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message