{"action":"created","issue":{"url":"https://api.github.com/repos/Brewtarget/brewtarget/issues/63","labels_url":"https://api.github.com/repos/Brewtarget/brewtarget/issues/63/labels{/name}","comments_url":"https://api.github.com/repos/Brewtarget/brewtarget/issues/63/comments","events_url":"https://api.github.com/repos/Brewtarget/brewtarget/issues/63/events","html_url":"https://github.com/Brewtarget/brewtarget/issues/63","id":97765627,"number":63,"title":"Final Batch Sparge always assuming 15min in Mash Wizard","user":{"login":"renato","id":3530,"avatar_url":"https://avatars.githubusercontent.com/u/3530?v=3","gravatar_id":"","url":"https://api.github.com/users/renato","html_url":"https://github.com/renato","followers_url":"https://api.github.com/users/renato/followers","following_url":"https://api.github.com/users/renato/following{/other_user}","gists_url":"https://api.github.com/users/renato/gists{/gist_id}","starred_url":"https://api.github.com/users/renato/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/renato/subscriptions","organizations_url":"https://api.github.com/users/renato/orgs","repos_url":"https://api.github.com/users/renato/repos","events_url":"https://api.github.com/users/renato/events{/privacy}","received_events_url":"https://api.github.com/users/renato/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-28T18:04:24Z","updated_at":"2015-08-03T15:20:16Z","closed_at":null,"body":"I created a mash configuration that has a \"Final Batch Sparge\" of 20 min. It is not a big deal, but even if I have a \"Final Batch Sparce\" with this duration, the wizard always assumes 15 min:\r\nhttps://github.com/Brewtarget/brewtarget/blob/master/src/MashWizard.cpp#L291\r\n\r\nAnother point that is not a big deal but sounded strange to me is that I changed the name \"Final Batch Sparge\" to \"Fly sparge\" (which is what I do) and the Mash Wizard ended up creating the step again:\r\nhttps://github.com/Brewtarget/brewtarget/blob/master/src/MashWizard.cpp#L251\r\n\r\nI think that it'd be interesting to set a mash step as the sparge step (a new checkbox or new infusion type) so the Mash Wizard wouldn't need to compare the step name with a hardcoded string.\r\n\r\nPlus, if a mash step for sparge is already present, its time should be used instead."},"comment":{"url":"https://api.github.com/repos/Brewtarget/brewtarget/issues/comments/127275055","html_url":"https://github.com/Brewtarget/brewtarget/issues/63#issuecomment-127275055","issue_url":"https://api.github.com/repos/Brewtarget/brewtarget/issues/63","id":127275055,"user":{"login":"theophae","id":11769613,"avatar_url":"https://avatars.githubusercontent.com/u/11769613?v=3","gravatar_id":"","url":"https://api.github.com/users/theophae","html_url":"https://github.com/theophae","followers_url":"https://api.github.com/users/theophae/followers","following_url":"https://api.github.com/users/theophae/following{/other_user}","gists_url":"https://api.github.com/users/theophae/gists{/gist_id}","starred_url":"https://api.github.com/users/theophae/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/theophae/subscriptions","organizations_url":"https://api.github.com/users/theophae/orgs","repos_url":"https://api.github.com/users/theophae/repos","events_url":"https://api.github.com/users/theophae/events{/privacy}","received_events_url":"https://api.github.com/users/theophae/received_events","type":"User","site_admin":false},"created_at":"2015-08-03T15:20:16Z","updated_at":"2015-08-03T15:20:16Z","body":"I also think that to add a checkbox to indicate the sparge step could be a good idea."}}