{"action":"created","issue":{"url":"https://api.github.com/repos/phstc/shoryuken/issues/85","labels_url":"https://api.github.com/repos/phstc/shoryuken/issues/85/labels{/name}","comments_url":"https://api.github.com/repos/phstc/shoryuken/issues/85/comments","events_url":"https://api.github.com/repos/phstc/shoryuken/issues/85/events","html_url":"https://github.com/phstc/shoryuken/pull/85","id":64994551,"number":85,"title":"Support ActiveJob queue name prefixes","user":{"login":"farski","id":3483,"avatar_url":"https://avatars.githubusercontent.com/u/3483?v=3","gravatar_id":"","url":"https://api.github.com/users/farski","html_url":"https://github.com/farski","followers_url":"https://api.github.com/users/farski/followers","following_url":"https://api.github.com/users/farski/following{/other_user}","gists_url":"https://api.github.com/users/farski/gists{/gist_id}","starred_url":"https://api.github.com/users/farski/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/farski/subscriptions","organizations_url":"https://api.github.com/users/farski/orgs","repos_url":"https://api.github.com/users/farski/repos","events_url":"https://api.github.com/users/farski/events{/privacy}","received_events_url":"https://api.github.com/users/farski/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-03-28T22:01:40Z","updated_at":"2015-07-15T23:10:04Z","closed_at":"2015-04-11T13:25:19Z","pull_request":{"url":"https://api.github.com/repos/phstc/shoryuken/pulls/85","html_url":"https://github.com/phstc/shoryuken/pull/85","diff_url":"https://github.com/phstc/shoryuken/pull/85.diff","patch_url":"https://github.com/phstc/shoryuken/pull/85.patch"},"body":"I took a shot at #57, because I was also looking for a more elegant solution than just adding the prefixes manually in the config file. Having a discrepancy between the queue name in the workers and the config seemed odd. ActiveJob seems to want the ultimately resolved names of queues to be transparent, so I think this should honor that.\r\n\r\nIt's pretty straightforward, but I honestly had no idea how to add tests that need a Rail environment to get loaded. I'm happy to write them if someone can point me in the right direction, though.\r\n\r\nBecause this would likely be a breaking change for anyone with a defined Active Job prefix, it must be opt-in, so I added a configuration that could be set in an initializer:\r\n\r\n```\r\n# config/initializers/shoryuken.rb\r\nShoryuken.active_job_queue_name_prefixing = true\r\n```"},"comment":{"url":"https://api.github.com/repos/phstc/shoryuken/issues/comments/121773230","html_url":"https://github.com/phstc/shoryuken/pull/85#issuecomment-121773230","issue_url":"https://api.github.com/repos/phstc/shoryuken/issues/85","id":121773230,"user":{"login":"farski","id":3483,"avatar_url":"https://avatars.githubusercontent.com/u/3483?v=3","gravatar_id":"","url":"https://api.github.com/users/farski","html_url":"https://github.com/farski","followers_url":"https://api.github.com/users/farski/followers","following_url":"https://api.github.com/users/farski/following{/other_user}","gists_url":"https://api.github.com/users/farski/gists{/gist_id}","starred_url":"https://api.github.com/users/farski/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/farski/subscriptions","organizations_url":"https://api.github.com/users/farski/orgs","repos_url":"https://api.github.com/users/farski/repos","events_url":"https://api.github.com/users/farski/events{/privacy}","received_events_url":"https://api.github.com/users/farski/received_events","type":"User","site_admin":false},"created_at":"2015-07-15T23:10:04Z","updated_at":"2015-07-15T23:10:04Z","body":"@phstc Thanks!"}}