[Sugar-devel] Review: [PATCH] Downgrading activities not allowed. (#2164)

[Manusheel Gupta]

Sorry for the delayed response on this thread.

On Sat, Oct 16, 2010 at 3:09 AM, Sascha Silbe <
sascha-ml-reply-to-2010-3 at silbe.org> wrote:

> Excerpts from Manusheel Gupta's message of Sat Oct 09 04:46:02 +0200 2010:
>
> > Ishan tried to add the tag - "Coauthored by" in his patch, and that line
> > gets scrubbed off.
>
> Have you added it right at the end, like you would do with the
> Reviewed-By: tag? E.g.:
>
> (assuming author is set to Shanjit Singh Jajmann <shanjit at seeta.in>)
> === Begin ===
> Allow downgrading activities (SL#2164)
>
> If the user tries to start an XO bundle containing a version of an activity
> that is older than the currently installed one, an option to downgrade the
> activity will be presented in the form of a confirmation alert.
>
> Co-Authored-By: Anubhav Aggarwal <anubhav at seeta.in>
> === End ===
>
>
Sascha, appreciate your continued guidance in patch submission procedures
and reviews. This format guideline has been Implemented in all the latest
patch submissions. Thank you for the timely feedback.



>
> PS: Please subscribe to the non-digest version of sugar-devel so you
> don't break threads. If there's a particular reason you're using the
> digest I can try to suggest an alternative.
>
>
My apologies for breaking the thread. I am subscribed to the non-digest
(individual e-mails) version only.

Manu



> Sascha
>
> --
> http://sascha.silbe.org/
> http://www.infra-silbe.de/
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20101022/0acac555/attachment.html>