{"push_id":690810169,"size":4,"distinct_size":4,"ref":"refs/heads/permissions-context-factory","head":"4fb05307e3495b2c1cca5ec343f7d1b77b7aa6b3","before":"7a137ccd700c3ecf80429d3b58ea3a7c7c118267","commits":[{"sha":"99e8a2f3efded7543b94565fd4bf20b289d6b447","author":{"email":"37ac5e111a9b2f779e373f78efa4f7678b93feb1@notmyidea.org","name":"Alexis Métaireau"},"message":"Use a partial for context.has_permission.\n\nThis avoids passing around information to the authorization policy when it's\nnot necessary. Here, at the definition time, the function is curried with the\nknown arguments.\n\nFrom a review by @leplatrem.","distinct":true,"url":"https://api.github.com/repos/mozilla-services/cliquet/commits/99e8a2f3efded7543b94565fd4bf20b289d6b447"},{"sha":"9a0cfc8d81d0681148c59f60e4b396120cbf8bdc","author":{"email":"37ac5e111a9b2f779e373f78efa4f7678b93feb1@notmyidea.org","name":"Alexis Métaireau"},"message":"Rename has_permission in check_permission.\n\nThe former could be a boolean flag. Here it's explicit it is doing an\noperation.\n\nFrom a review by @leplatrem","distinct":true,"url":"https://api.github.com/repos/mozilla-services/cliquet/commits/9a0cfc8d81d0681148c59f60e4b396120cbf8bdc"},{"sha":"49ab1eaa738cd9f49791b029036310766b522168","author":{"email":"37ac5e111a9b2f779e373f78efa4f7678b93feb1@notmyidea.org","name":"Alexis Métaireau"},"message":"Rename ctx.object_type into ctx.resource_name","distinct":true,"url":"https://api.github.com/repos/mozilla-services/cliquet/commits/49ab1eaa738cd9f49791b029036310766b522168"},{"sha":"4fb05307e3495b2c1cca5ec343f7d1b77b7aa6b3","author":{"email":"37ac5e111a9b2f779e373f78efa4f7678b93feb1@notmyidea.org","name":"Alexis Métaireau"},"message":"Use a constant for authorization.DYNAMIC.\n\nRather than using a string, which is error prone, having a constant allows to\ncatch errors earlier in the dev process.","distinct":true,"url":"https://api.github.com/repos/mozilla-services/cliquet/commits/4fb05307e3495b2c1cca5ec343f7d1b77b7aa6b3"}]}