Yes, we fixed the issue in our code, it should be ok now. At the moment, when I try to run all the tests I get some errors from other packages. I tried to fix them, but I don't know how to do that, since they come from modules and packages dealing with repositories, and they are unrelated to our code. Thank you very much! Stefano
2013/2/21 Tim Van Steenburgh <[email protected]> > Hi Simone and Stefano, > > I pulled your si/5453 branch today to do some more testing. I rebased it > against master and kicked off the tests (`./run_tests` in the project root) > and got some failures. If you guys can get those fixed up and pushed back, > I'll try to wrap up the testing. > > Thanks! > > Tim Van Steenburgh > > > On Monday, February 18, 2013 at 2:26 PM, Simone Gatti wrote: > > > Dear Cory, > > what Dave said is correct, we refer to user stats. > > I'm sorry for this ambiguity, that we didn't noticed before your comment. > > We welcome your *feedback on user stats*, since we have not yet received > > many comments yet. > > > > Simone > > > > > > 2013/2/18 Dave Brondsema <[email protected] (mailto:[email protected] > )> > > > > > Cory, I think Stefano is referring to user stats, which his feature > branch > > > starts collecting, not user profile data (gender, location, etc). > > > > > > But on the topic of user profiel data, we've had a least one > SourceForge > > > user > > > communicate to use that he/she thought the fields were required. I can > > > see how > > > this might be inferred since they're the first thing you see on the > > > /auth/prefs/ > > > form. We might consider labelling those optional, or putting them on a > > > separate > > > page from "subscriptions" and other sections on that page. > > > > > > -Dave > > > > > > > > > > > > On 2/18/13 9:55 AM, Cory Johns wrote: > > > > Stefano, > > > > > > > > Could a user simply not fill in the personal info fields they don't > wish > > > to > > > > share? What is the value of entering that info but then not > displaying > > > > > > it; > > > > to encourage users to enter it if only for our edification? > > > > > > > > > > > > Thanks, > > > > > > > > Cory > > > > > > > > > > > > On Sat, Feb 16, 2013 at 9:53 AM, Stefano Invernizzi < > > > > [email protected] (mailto: > [email protected])> wrote: > > > > > > > > > Dear all, > > > > > > > > > > I recently pushed some new commits allowing a single user to hide > his or > > > > > her personal statistics. I and Simone implemented it since some > users > > > > > > > > > > > > > > > > > > > may > > > > > prefer not to show this data. In that case, data is still > available for > > > > > their personal use. However, if you think we should not allow > users to > > > > > > > > > > > > > > > do > > > > > this, we can simply put it back as it was. > > > > > As usual, we hope to get some feedbacks from you about this, as > well as > > > > > about the rest of submitted code. > > > > > It would be great for us if the code could be reviewed and, if you > > > > > > > > > > > > > > > think it > > > > > would be useful, included on the forge before we complete the > thesis we > > > > > > > > > > are > > > > > working on. > > > > > Thank you very much! > > > > > Stefano > > > > > > > > > > > > > > > > > > > > > > > > -- > > > Dave Brondsema : [email protected] (mailto:[email protected]) > > > http://www.brondsema.net : personal > > > http://www.splike.com : programming > > > <>< > > > > > > > > > > > >
