Closed ngiachou closed 5 years ago
Test comment from slack. View in Slack
The main app functionality looks good to me. My only suggestion is that we should add an extra option that exits the program at every stage of the app.
By exit you mean closing the app or more like a go-back functionality?
Maybe a clearScreen()
which so info that's not needed won't be printed on the console (example: previous messages)
I think Exit option should exist in every Menu and sub-menu. But Go back only in Sub-menus. For early development we should just implement the Exit option which terminates the app, and the Go back can just go to the Main Menu
@pagidas maybe we should open a new PR for your comments on Dec 15, 2018?
I see that part of the issue has been solved. But not all, am I right? So I suppose you mean open a new PR to solve and probably close the issue. If so, then do it
* Maybe a `clearScreen()` which so info that's not needed won't be printed on the console (example: previous messages) * I think **Exit** option should exist in every Menu and sub-menu. But **Go back** only in Sub-menus. For early development we should just implement the **Exit** option which terminates the app, and the **Go back** can just go to the Main Menu
I was referring to this comment, but as I see those features are also implemented.
Nevertheless, we should keep this issue open, since a lot more functionality is to be implemented. We can observe that from the numerous TODO comments throughout mainApp.py
.
Closing. This issue is totally out of context because we don't have any console functionality now.
The main app will consist of three modes which depend on the opening time of the app. The main app will be a terminal app for now. Later we will implement the GUI staff.
For the first time the app is launched, the following should happen in terminal:
Example run:
If the app is launched for the first time in a month, the following should happen in terminal:
Example run:
For any other case a main menu will be shown.
where the statistics option has sub-options.
Example run: