{"push_id":2348022251,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"502fefde45eb2cd16f938e5ccb63e4bb62bbea56","before":"1d6cf39dec83e5aa9ffb70839c47efc94743325e","commits":[{"sha":"502fefde45eb2cd16f938e5ccb63e4bb62bbea56","author":{"name":"Israel Madueme","email":"9eafe19b647b94156d9b449f02b68e88a3c578cd@gmail.com"},"message":"revisions: use active diff id if diff id in url is None\n\nSummary:\nWhen a user visits /revisions/D1 lando-ui queries lando-api to get the\nrevision information expecting to get back diff info for the most recent\ndiff. On the other hand if the user visits /revisions/D1/99, they will\nsee diff information for diff 99 specifically.\n\nPart of loading the revision page includes querying the dryrun endpoint\nwhich requires a diff id. This doesn't happen if you're not logged in.\n\nThese two points made this bug hard to hit when doing local development:\nthe UI would crash in the case that the user is logged in and visits a\nurl without requesting a specific diff.\n\nThis commit fixes this problem to default to the diff provided in the\nrequested revision info if there is none in the url. (If there is one\nin the url, the revision info diff and the url diff would be equal\nby nature of how the get_revision query is made).\n\nTest Plan: invoke test, try it out!\n\nReviewers: smacleod\n\nReviewed By: smacleod\n\nBug #: 1437542\n\nDifferential Revision: https://phabricator.services.mozilla.com/D622","distinct":true,"url":"https://api.github.com/repos/mozilla-conduit/lando-ui/commits/502fefde45eb2cd16f938e5ccb63e4bb62bbea56"}]}
{"actor":{"display_login":"purelogiq"}}