-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add temperature offset #51
Labels
Comments
Looking at the existing UI, where do you think such a setting belongs? Describe the journey you'd expect to take as a user wanting to specify a 'constant offset' (let's not get too fancy here...) |
I can see 2 ways of doing it.
one of just putting a constant offset on the recipe creation screen and can update a recipe automatically, or
Put a text box to be under the roast button that is non-volatile (and also settable by command line argument 😊)
I would choose the second, just in case I went through a lot of roasters.
From: int3ll3ct<mailto:[email protected]>
Sent: Tuesday, March 28, 2017 6:14 PM
To: Roastero/Openroast<mailto:[email protected]>
Cc: Craig Romei<mailto:[email protected]>, Author<mailto:[email protected]>
Looking at the existing UI, where do you think such a setting belongs? Describe the journey you'd expect to take as a user wanting to specify a 'constant offset' (let's not get too fancy here...)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#51 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AYp8r3J8Qfj2KRqVFQyLA9nh66RZoGUWks5rqbBhgaJpZM4Mkdz5>.
|
I agree that either an edit box near the roast button, or an edit box in the recipe browser window, would be the best places for this. Thanks. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
For those of us with cool running roasters, it would be awesome to have a way of just setting an offset so when I download recipes, I wouldn't have to change every temperature.
The text was updated successfully, but these errors were encountered: