<div dir="ltr"><br><br><div class="gmail_quote">On Thu, Jul 17, 2008 at 5:31 AM, Tomeu Vizoso <<a href="mailto:tomeu@tomeuvizoso.net">tomeu@tomeuvizoso.net</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="Ih2E3d">On Thu, Jul 17, 2008 at 3:19 AM, Eben Eliason <<a href="mailto:eben.eliason@gmail.com">eben.eliason@gmail.com</a>> wrote:<br>
> Yeah, I'm not sure that this is expected behavior (I would expect not). The<br>
> intent is that a "customization" upgrade would allow additive changes to the<br>
> favorites ring, for instance to allow a school to ensure that every kid has<br>
> brand new activity X in the ring on the first day of class (if the kids<br>
> later remove it, that's up to them, of course). The question, then, is<br>
> if/why installing that RPM behaved in the manner of a customization instead<br>
> of a basic software update.<br>
> Tomeu, do you know how this is expected to work?<br>
<br>
</div>When kids update to a version that has the notion of favorites, we<br>
don't want to show the favorites screen empty. That's why we added<br>
code for adding the activities mentioned in activities.default.<br></blockquote><div><br></div><div>I'm OK with this if it's a one-time change. In other words, it should check for an empty favorites list and apply the defaults if none exist.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
Also, when you update to a newer version, favorites.default might<br>
change so we merge it again with the users favorites. We don't know if<br>
an activity present in activities.default but not in the favorites was<br>
removed by the user or added by the school/deployer.<br></blockquote><div><br></div><div>Well, it depends on the use case. Is editing favorites.default the intended way for a country or school to manage updates? If so, then I suppose that this is correct. If not, then we should really only be setting the favorites from the default file when no favorites are set at all (should only happen once), and otherwise do a merge from a favorites list on a customization key instead.</div>
<div><br></div><div>- Eben </div></div></div>