{"action":"created","issue":{"url":"https://api.github.com/repos/moneyadviceservice/rad_consumer/issues/298","repository_url":"https://api.github.com/repos/moneyadviceservice/rad_consumer","labels_url":"https://api.github.com/repos/moneyadviceservice/rad_consumer/issues/298/labels{/name}","comments_url":"https://api.github.com/repos/moneyadviceservice/rad_consumer/issues/298/comments","events_url":"https://api.github.com/repos/moneyadviceservice/rad_consumer/issues/298/events","html_url":"https://github.com/moneyadviceservice/rad_consumer/pull/298","id":145651495,"number":298,"title":"Always update the search url cache in the session store","user":{"login":"munckymagik","id":306583,"avatar_url":"https://avatars.githubusercontent.com/u/306583?v=3","gravatar_id":"","url":"https://api.github.com/users/munckymagik","html_url":"https://github.com/munckymagik","followers_url":"https://api.github.com/users/munckymagik/followers","following_url":"https://api.github.com/users/munckymagik/following{/other_user}","gists_url":"https://api.github.com/users/munckymagik/gists{/gist_id}","starred_url":"https://api.github.com/users/munckymagik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/munckymagik/subscriptions","organizations_url":"https://api.github.com/users/munckymagik/orgs","repos_url":"https://api.github.com/users/munckymagik/repos","events_url":"https://api.github.com/users/munckymagik/events{/privacy}","received_events_url":"https://api.github.com/users/munckymagik/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-04-04T11:01:47Z","updated_at":"2016-04-04T15:24:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/moneyadviceservice/rad_consumer/pulls/298","html_url":"https://github.com/moneyadviceservice/rad_consumer/pull/298","diff_url":"https://github.com/moneyadviceservice/rad_consumer/pull/298.diff","patch_url":"https://github.com/moneyadviceservice/rad_consumer/pull/298.patch"},"body":"# What?\r\n\r\nWe noticed a bug where the back link in the firm profile page wasn't returning us to the most recent search. This change fixes that.\r\n\r\n# Why?\r\n\r\nThere is a guard condition that used to short-circuit the `#store` method if the firm being visited was already in the list of most recently visited firms.\r\n\r\nUnfortunately update of the most recent search URL was happening after this guard condition.\r\n\r\nSo this change separates out the logic for the two concerns so they cannot affect each other anymore."},"comment":{"url":"https://api.github.com/repos/moneyadviceservice/rad_consumer/issues/comments/205348159","html_url":"https://github.com/moneyadviceservice/rad_consumer/pull/298#issuecomment-205348159","issue_url":"https://api.github.com/repos/moneyadviceservice/rad_consumer/issues/298","id":205348159,"user":{"login":"tomas-stefano","id":27509,"avatar_url":"https://avatars.githubusercontent.com/u/27509?v=3","gravatar_id":"","url":"https://api.github.com/users/tomas-stefano","html_url":"https://github.com/tomas-stefano","followers_url":"https://api.github.com/users/tomas-stefano/followers","following_url":"https://api.github.com/users/tomas-stefano/following{/other_user}","gists_url":"https://api.github.com/users/tomas-stefano/gists{/gist_id}","starred_url":"https://api.github.com/users/tomas-stefano/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomas-stefano/subscriptions","organizations_url":"https://api.github.com/users/tomas-stefano/orgs","repos_url":"https://api.github.com/users/tomas-stefano/repos","events_url":"https://api.github.com/users/tomas-stefano/events{/privacy}","received_events_url":"https://api.github.com/users/tomas-stefano/received_events","type":"User","site_admin":false},"created_at":"2016-04-04T15:24:54Z","updated_at":"2016-04-04T15:24:54Z","body":"LGTM :+1: "}}