Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

2015-08-25 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37761/#review96377 --- Ship it! Ship It! - Zameer Manji On Aug. 25, 2015, 8:01 a.m.,

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

2015-08-25 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37761/#review96362 --- Ship it! Ship It! - Maxim Khutornenko On Aug. 25, 2015, 3:01

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

2015-08-25 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37761/#review96399 --- Master (86a547b) is green with this patch.

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

2015-08-25 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37761/#review96394 --- What if they click on an old update and the instance page doesn't

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

2015-08-25 Thread Joshua Cohen
On Aug. 25, 2015, 6:42 p.m., David McLaughlin wrote: What if they click on an old update and the instance page doesn't reflect the change made in this instance event? Do we care? I don't think we care, there's not much we can do in that case, is there? The only thing I can think of is

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

2015-08-25 Thread Joshua Cohen
On Aug. 25, 2015, 6:42 p.m., David McLaughlin wrote: What if they click on an old update and the instance page doesn't reflect the change made in this instance event? Do we care? Joshua Cohen wrote: I don't think we care, there's not much we can do in that case, is there? The

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

2015-08-25 Thread Maxim Khutornenko
On Aug. 25, 2015, 6:42 p.m., David McLaughlin wrote: What if they click on an old update and the instance page doesn't reflect the change made in this instance event? Do we care? Joshua Cohen wrote: I don't think we care, there's not much we can do in that case, is there? The

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

2015-08-25 Thread David McLaughlin
On Aug. 25, 2015, 6:42 p.m., David McLaughlin wrote: What if they click on an old update and the instance page doesn't reflect the change made in this instance event? Do we care? Joshua Cohen wrote: I don't think we care, there's not much we can do in that case, is there? The

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

2015-08-25 Thread Joshua Cohen
On Aug. 25, 2015, 6:42 p.m., David McLaughlin wrote: What if they click on an old update and the instance page doesn't reflect the change made in this instance event? Do we care? Joshua Cohen wrote: I don't think we care, there's not much we can do in that case, is there? The

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

2015-08-25 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37761/#review96479 --- Ship it! Ship It! - David McLaughlin On Aug. 25, 2015, 3:01