> On 2012-04-23 14:47:28, maze wrote: > > This change broke our existing waves. After logging in, the inbox stayed > > empty and the following stack trace was immediately shown (replaced our > > domain with xxx.com): > > > > Token: 1335192295655 > > Class$NXf: Invalid id 'xxx.com/w+QAyhGJwY_YA': Wave id must be of the form > > <domain>!<id> > > Unknown.WF (Unknown source:0) > > Unknown.cG (Unknown source:0) > > Unknown.KWb (Unknown source:0) > > Unknown.CYf (Unknown source:0) > > Unknown.Jqc (Unknown source:0) > > Unknown.Kqc (Unknown source:0) > > Unknown.Gqc (Unknown source:0) > > Caused by: Token: 1335192295655 > > Invalid id 'xxx.com/w+QAyhGJwY_YA': Wave id must be of the form > > <domain>!<id> > > Unknown.WF (Unknown source:0) > > Unknown.PXf (Unknown source:0) > > Unknown.UXf (Unknown source:0) > > Unknown.CYf (Unknown source:0) > > Unknown.Jqc (Unknown source:0) > > Unknown.Kqc (Unknown source:0) > > Unknown.Gqc (Unknown source:0) > > > > > > I had to revert the changes in order get our waves back. We have no special > > robots or gadgets in use. > > > > Is this intended? Is there a way to "migrate" the old style waves?
Did you try to clean the browser cache? - Yuri ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4828/#review7130 ----------------------------------------------------------- On 2012-04-20 12:49:04, Yuri Zelikov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4828/ > ----------------------------------------------------------- > > (Updated 2012-04-20 12:49:04) > > > Review request for wave, Michael MacFadden and Ali Lown. > > > Summary > ------- > > Changes WaveId representation to the modern style: using '/' as separator > instead of '!'. This is required for consistency as most of the WIAB already > uses the modern '/' separator while the Robot API still returns old waveId > representation. > The change might break some robots that were built for WIAB, but I don't > think there are too many yet. > > > Diffs > ----- > > src/com/google/wave/api/ApiIdSerializer.java 674f235 > src/org/waveprotocol/wave/model/id/WaveId.java 45c0d6f > > Diff: https://reviews.apache.org/r/4828/diff > > > Testing > ------- > > Checked that all test pass. > Verified the web client works. > > > Thanks, > > Yuri > >