Feature: Add Windchill application specification #5
Conversation
Add Timezone Slackbot app idea Resolves: N/a See also: N/a
…o feature/add-tz-bot Refactor: Resolve merge conflict with `master` branch
Add Windchill application specification Resolves: N/a See also: N/a
Feature/add tz bot
Projects/Windchill-App.md
Outdated
| - [ ] User can enter the actual temperature (Celcius) and the wind speed | ||
| (kilometers per hour) | ||
| - [ ] User can press the `Calculate` button to display the wind chill | ||
| - [ ] User will receive and error message if data values are not entered |
There was a problem hiding this comment.
Hmm.. "error message if ... or if the resulting wind chill factor is greater..."
Is this also an error message? Shouldn't this be the actual result and not an error? 🤔
There was a problem hiding this comment.
For more advanced developers this would actually be an assertion since by definition, wind chill factors must be less than the actual temperature. However, since this is a beginner app I didn't want to introduce assert so I left chose to specify it as an error. It really signifies there's a problem with the calculation if this case is true.
There was a problem hiding this comment.
Oh, I see... Thanks for the clarification!
Projects/Windchill-App.md
Outdated
| - [ ] User will receive and error message if data values are not entered | ||
| or if the resulting wind chill factor is greater than or equal to the | ||
| actual temperature. | ||
| - [ ] User will be prompted to enter new data values if `Calculate` is |
There was a problem hiding this comment.
Maybe move this in to the bonus features and the English metric to be a User story as it seems much more of a "must" to have metric system also... What do you say? 🤔
There was a problem hiding this comment.
Agree. I left the basic requirement to edit for valid input as a requirement, but moved the other two to bonus features.
Projects/Windchill-App.md
Outdated
|
|
||
| ## Bonus features | ||
|
|
||
| - [ ] User can select the measurement system calculations will be performed |
There was a problem hiding this comment.
Other bonus feature: User can see the wind chill factor field automatically changed when both fields have valid data or something like that -> this means that the user doesn't have to press the button always to perform the calculation. It could be done on the onChange event on the inputs,
There was a problem hiding this comment.
Agreed. Good idea...great way to teach event handling.
…17/app-ideas into feature/add-wind-chill Refactor: Resolve merge conflicts with `master` branch
Implement suggestions from PR review - See [PR comments](#5) for details Resolves: N/a See also: N/a
Add Windchill application specification
Resolves: N/a
See also: N/a