{"shas":[["a9af9615290fe9f7e8ad992df0fc80b5f1edd0f7","c3918f39022407dee6a0056ceca9b1aa80e8e4ce@terrorware.com","Refactored the user profile fetching code in the core.views.topics()\n\nI also cleaned up some of the line lengths and added a sketch for how to pass a list\nof comments that the user has voted on can be passed to the template.  I still have\nto implement the method that generates this list.","Geoffrey Hing"],["df7bde5d0df4d63c7d915e6b3cdb074297cdc0fa","c3918f39022407dee6a0056ceca9b1aa80e8e4ce@terrorware.com","Created bare-minimum tests for testing core.Topic.user_voted_comments()","Geoffrey Hing"],["4e529018e4d9ff449976fca2d14dbcb0d6a58b86","c3918f39022407dee6a0056ceca9b1aa80e8e4ce@terrorware.com","Added (commented out) code to show how I would pass a list of comments that a user had responded to, to the template.","Geoffrey Hing"],["40f3caec09439f68aecfba6b75cde6cdf4126911","c3918f39022407dee6a0056ceca9b1aa80e8e4ce@terrorware.com","First pass at implementing methods to return a list of comments that a user had responded to.","Geoffrey Hing"],["d63ba8cc44a53e356b649fea36121e22886fd633","c3918f39022407dee6a0056ceca9b1aa80e8e4ce@terrorware.com","Fixed minor bug in core.Topic.user_voted_comments()\n\nI forgot to pass the user_profile argument to the call to user_responded_comments()","Geoffrey Hing"],["5d2a0a39782b88a5b3a7aa28fb5bfd3ced01dd88","c3918f39022407dee6a0056ceca9b1aa80e8e4ce@terrorware.com","First pass at core.Topics.user_responded_comments()","Geoffrey Hing"],["d323f7b53c7b325c1fb1d82e43dcf9e8bc31ae4a","c3918f39022407dee6a0056ceca9b1aa80e8e4ce@terrorware.com","Made core.Topic methods returning information on which comments users have responded to return lists of ids rather than objects.\n\nThis change in functionality comes because I initially misunderstood how the values_list() method of a QuerySet works.  From the Django docs:\n\n\"If you have a field called foo that is a ForeignKey, the default values() call will return a dictionary key called foo_id, since this is the name of the hidden model attribute that stores the actual value (the foo attribute refers to the related model).\"\n\nIt caused the methods to be renamed as well.","Geoffrey Hing"],["731274a76860d54caa3fa78153c388b46808f16e","c3918f39022407dee6a0056ceca9b1aa80e8e4ce@terrorware.com","Enabled revealing a list of ids of comments that a user has commented on to the topic view template.","Geoffrey Hing"],["2c62bb4313e4b71161a1d5f64fa17aadc8998ebc","c3918f39022407dee6a0056ceca9b1aa80e8e4ce@terrorware.com","Merge remote branch 'origin/master' into geoff","Geoffrey Hing"],["b463b298c20b554579e05cb617f1e86438d95b14","c3918f39022407dee6a0056ceca9b1aa80e8e4ce@terrorware.com","Merge branch 'master', remote branch 'origin'","Geoffrey Hing"],["f7d7cd04780b420c9ea64f55b69a4221231cf0b9","c3918f39022407dee6a0056ceca9b1aa80e8e4ce@terrorware.com","Merge branch 'geoff'","Geoffrey Hing"]],"ref":"refs/heads/master","push_id":20928449,"head":"f7d7cd04780b420c9ea64f55b69a4221231cf0b9","size":11}