-
Notifications
You must be signed in to change notification settings - Fork 4
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 task verifier display to ui and all related interfaces #155
Conversation
a83c964
to
07b93f1
Compare
Did not add any changes in the force push. Just trying to get travis to run on the PR |
@@ -101,8 +101,8 @@ public void close() throws IOException { | |||
|
|||
@Override | |||
public void write(final int character) throws IOException { | |||
char symbol = (char) character; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please try to avoid unrelated changes in the future.
I think it is better to just open a separate PR for these. (It will be small and reviewed+merged quickly)
The problem is: Now blame will show a commit, which should have nothing to do with this class.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But you can leave it in this PR now.
This PR adds a task status display to the ui as a task status tab next to the console tab.
The task verification status needs to be refreshed manually using the refresh button.
The task information model allows nested tasks with their own verification status.
Closes #144
Related to #135