{"push_id":1441150211,"size":8,"distinct_size":1,"ref":"refs/heads/master","head":"c268b5c09743f2bc8d8f624e1146d82140fe8cc2","before":"353b65bd5e9a4580dedf74cc90e4923f1839c8f3","commits":[{"sha":"277e087490c3d0d5885c00fd8f02c4c0bc9d45a1","author":{"name":"Kevin Dew","email":"cce05575eed51e8191034eb5c9f24c2a732ece47@digital.cabinet-office.gov.uk"},"message":"Retrieve items that are linked to a content item\n\nTo be used as part of dependency resolution to determine which items to\nupdate.","distinct":false,"url":"https://api.github.com/repos/alphagov/publishing-api/commits/277e087490c3d0d5885c00fd8f02c4c0bc9d45a1"},{"sha":"0f2cd933d4ae953e2692b466fd4887c5a1e3b1e0","author":{"name":"Kevin Dew","email":"cce05575eed51e8191034eb5c9f24c2a732ece47@digital.cabinet-office.gov.uk"},"message":"Don't require creating link set for helper\n\nAllows us to save a few extra method calls in tests.","distinct":false,"url":"https://api.github.com/repos/alphagov/publishing-api/commits/0f2cd933d4ae953e2692b466fd4887c5a1e3b1e0"},{"sha":"8c130e8cb677642278eb23bd63c4bacd8437e4da","author":{"name":"Kevin Dew","email":"cce05575eed51e8191034eb5c9f24c2a732ece47@digital.cabinet-office.gov.uk"},"message":"Refactor ContentDependencies query class\n\nThis is a re-approach to ContentDependencies to make it a class that can\nact as a one method call approach to access all the content_ids and\nlocales that require an update as part of dependency resolution. The\nmajor pro for this being that it can be a place that a single set of\ntests are wrote to test dependency resolution is working as expected.\n\nHopefully this has also made the process of dependency resolution easier\nto understand.","distinct":false,"url":"https://api.github.com/repos/alphagov/publishing-api/commits/8c130e8cb677642278eb23bd63c4bacd8437e4da"},{"sha":"3d6b335fe8e7096e67a2eab0d3f5d2c8a753fbfb","author":{"name":"Kevin Dew","email":"cce05575eed51e8191034eb5c9f24c2a732ece47@digital.cabinet-office.gov.uk"},"message":"Drop fields in dependency resolution\n\nIt is not currently being used for anything and would require a number\nof changes for it to become useful.","distinct":false,"url":"https://api.github.com/repos/alphagov/publishing-api/commits/3d6b335fe8e7096e67a2eab0d3f5d2c8a753fbfb"},{"sha":"b213afa1a198f7676e6f8e85c15d22c0cd558589","author":{"name":"Kevin Dew","email":"cce05575eed51e8191034eb5c9f24c2a732ece47@digital.cabinet-office.gov.uk"},"message":"Dual purpose LocalesForContentItem query\n\nThis change allows us to call Queries::LocalesForContentItem with either\na `.for_one` or `.for_many` method allow you to get the information for\na single content_id or multiple content_ids respectively.\n\nQueries::LocalesForContentItem.for_one will return an array of locales:\n```\nQueries::LocalesForContentItem.for_one(\"5f5bf416-7631-11e4-a3cb-005056011aef\")\n=> [\"en\", \"fr\"]\n```\n\nQueries::LocalesForContentItem.for_many will return as a hash of\ncontent_ids with locales:\n```\nQueries::LocalesForContentItem.for_many([\"5f5bf416-7631-11e4-a3cb-005056011aef\", \"5eefad99-7631-11e4-a3cb-005056011aef\"])\n=> [[\"5f5bf416-7631-11e4-a3cb-005056011aef\", \"en\"], [\"5f5bf416-7631-11e4-a3cb-005056011aef\", fr\"], [\"5eefad99-7631-11e4-a3cb-005056011aef\" \"en\"], [\"5eefad99-7631-11e4-a3cb-005056011aef\", \"es-419\"]]\n```","distinct":false,"url":"https://api.github.com/repos/alphagov/publishing-api/commits/b213afa1a198f7676e6f8e85c15d22c0cd558589"},{"sha":"bc7e0c7909e16e05aacf85e24f4b1de65270a63e","author":{"name":"Kevin Dew","email":"cce05575eed51e8191034eb5c9f24c2a732ece47@digital.cabinet-office.gov.uk"},"message":"Remove classes no longer used","distinct":false,"url":"https://api.github.com/repos/alphagov/publishing-api/commits/bc7e0c7909e16e05aacf85e24f4b1de65270a63e"},{"sha":"e423284e1c9ae95ac171fa53f49f8478dd911709","author":{"name":"Kevin Dew","email":"cce05575eed51e8191034eb5c9f24c2a732ece47@digital.cabinet-office.gov.uk"},"message":"Single Queries::LocalesForContentItems query\n\nAs we don't use the `for_one` method on Queries::LocalesForContentItem\nit makes more sense that we drop this method and only have the\n`for_many` one.\n\nHowever without the context of `for_one` the `for_many` method seems\na confusing name. So Queries::LocalesForContentItem has been renamed to\nQueries::LocalesForContentItems and the `for_many` method has been\nrenamed to use the `call` convention that is across our Query classes\n(call is still not a great method name, so suggestions of improvements\nwelcome ☺️)","distinct":false,"url":"https://api.github.com/repos/alphagov/publishing-api/commits/e423284e1c9ae95ac171fa53f49f8478dd911709"},{"sha":"c268b5c09743f2bc8d8f624e1146d82140fe8cc2","author":{"name":"Kevin Dew","email":"6d7ab0ab939d51d611fa8f8a1e87387a1f13395e@me.com"},"message":"Merge pull request #624 from alphagov/refactor-dependency-linking\n\nRefactor Dependency Resolution","distinct":true,"url":"https://api.github.com/repos/alphagov/publishing-api/commits/c268b5c09743f2bc8d8f624e1146d82140fe8cc2"}]}
{"actor":{"display_login":"kevindew"}}